Skip to content

Conversation

@beytarovski
Copy link

@beytarovski beytarovski commented Nov 10, 2024

@beytarovski beytarovski requested a review from dmythro as a code owner November 10, 2024 13:10
@dmythro
Copy link
Member

dmythro commented Oct 20, 2025

Hi @beytarovski. Sorry for the delay. I did check this, and feels like the name should be Turkiye in English as there's no ü in English alphabet. I did read the discussion and there's no strict "pro" for using ü but Turkiye is good enough for consistency (and probably screen readers etc). The name is strictly English in this library and native was updated to Türkiye a long time ago.

Any other thoughts on this? Feels like Turkiye in English is the most consistent solution here. But it also feels like it's best to be updated with a feature like #35 so country is easily found by its old name and no breaking changes for existing code which uses something like getCountryCode util.

@beytarovski
Copy link
Author

beytarovski commented Oct 27, 2025

Hi, thanks for reviewing this, dear dmythro!

Türkiye is not a native word only. It’s an English word right now. I think it's the only word in English that contains a foreign letter.

✅ The US government also uses Türkiye in its English publications: https://www.state.gov/releases/2025/09/the-reopening-of-the-iraq-turkiye-pipeline/
✅ It's standardized by ISO 3166.

I'd go with Türkiye with eyes closed, which is already an English word.

If you're not 100% sure, you may also consider Turkey (Türkiye) until it's spread enough 👉 see here.

But, Turkiye (without "ü") doesn't mean anything either for English nor Turkish natives.

❌ Keeping only Turkey is just wrong. It's the same as calling "Istanbul" Constantinople. Calling Istanbul as Constantinople by some people won't change the fact ;)

👉English Prime Minister is also use Türkiye in his tweets: https://x.com/Keir_Starmer/status/1982853597718347840

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants