Skip to content

Rename ko.po back to ko_KR.po#11074

Merged
jcollie merged 1 commit intoghostty-org:mainfrom
00-kat:north-korea
Feb 28, 2026
Merged

Rename ko.po back to ko_KR.po#11074
jcollie merged 1 commit intoghostty-org:mainfrom
00-kat:north-korea

Conversation

@00-kat
Copy link
Contributor

@00-kat 00-kat commented Feb 28, 2026

While it was renamed from ko_KR.UTF-8.po to ko.po in #10976, @uhojin, a Korean locale maintainer, notes that “ko_KR [South Korean] makes more sense in locale context just to avoid any potential confusion between 한국어 vs 조선어”.

Despite ko_KP (North Korean) not being present in glibc (as of version 2.43), and the ISO639 maintainers expressing disapproval of ko_KP, it is possible opinions may change in the future, and individual opinions may be contested—disambiguating doesn't hurt.

Requesting a review from all involved parties; I wish you all opine before merging.

@00-kat 00-kat requested review from jcollie, trag1c and uhojin February 28, 2026 07:59
@00-kat 00-kat added the translations Translations of the Ghostty app label Feb 28, 2026
@00-kat 00-kat mentioned this pull request Feb 28, 2026
While it was renamed from ko_KR.UTF-8.po to ko.po in #10976, @uhojin,
a Korean locale maintainer, notes [1] that “ko_KR [*South* Korean] makes
more sense in locale context just to avoid any potential confusion
between 한국어 vs 조선어”.

Despite ko_KP (North Korean) not being present in glibc (as of version
2.43), and the ISO639 maintainers expressing disapproval of ko_KP [2],
it is possible opinions may change in the future, and individual
opinions may be contested—disambiguating doesn't hurt.

[1]: #10976 (comment)
[2]: #10976 (comment)
@jcollie jcollie merged commit 14166bc into ghostty-org:main Feb 28, 2026
62 checks passed
@github-actions github-actions bot added this to the 1.3.0 milestone Feb 28, 2026
@00-kat 00-kat deleted the north-korea branch February 28, 2026 08:52
@trag1c trag1c removed their request for review February 28, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

translations Translations of the Ghostty app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants