Rewrite the localization teams docs to improve legibility, and explicitly permit members who know each other#12720
Rewrite the localization teams docs to improve legibility, and explicitly permit members who know each other#1272000-kat wants to merge 1 commit into
Conversation
pluiedev
left a comment
There was a problem hiding this comment.
The exclamation marks were indeed a bit excessive but hey
7e40ac2 to
d2c7310
Compare
|
Figured out a nice way to phrase the other parts of the old text that irked me. |
78b10c2 to
351799f
Compare
| 3. Locale maintainers are _informed of updates_ to the translations they | ||
| maintain via GitHub's review requests. | ||
|
|
||
| That's all there is to it, really. If you wish, you may leave your localization |
There was a problem hiding this comment.
“That's all there is to it, really.” is a pretty strong way of saying the above list is exhaustive; I don't think there's any other things about locale maintainers, but I might be wrong.
There was a problem hiding this comment.
I feel like mentioning that a maintainer is allowed to only deal with updates, or only review, would be helpful, since currently the list is more precise than reality. I'm not sure how to word it though, since there's also the extra “there must be at least one reviewer and at least one updater” restriction.
Maybe I have too many exclamation marks, let me know if I should metaphorically calm down.Fixed now.My wording is intentionally biased toward languages spoken less (edit: not nearly as much anymore), but I specifically do not disallow members who know each other regardless of language popularity. Quoting myself from Discord1:
I don't believe it's necessarily worse, and if you have more than two members then the issue greatly diminishes too, but I don't want people to see this and go “oh no I need to get my translations in before Ghostty 1.4 that releases while I'm sleeping tomorrow so I should ask my bestie to help”, and to instead be willing to be more patient, at least for a reasonable amount of time (which I consider to be ≤ 2 months).
Footnotes
@trag1c and I chatted about this prior to this PR in
#maintainerson the Ghostty Discord server. If you have access to that channel, check out these links: 1, 2, 3. ↩