Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add 'USA' to the list of aliases #102

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

utilmind
Copy link

The list of aliases is obviously missing the USA. Please include! Thanks!

@alcohol
Copy link
Member

alcohol commented Apr 26, 2024

This PR does two things, and while I am reluctant but indifferent about the actual purpose, the additional changes I definitely do not accept.

@utilmind
Copy link
Author

utilmind commented Apr 26, 2024

@alcohol I can split this PR on two parts, which can be reviewed separately.

"Additional" changes allows lookup for country names like "Cote d’Ivoire" with regular ASCII characters, without symbols like "ô" (I don't even have this character in my keyboard layout).

@utilmind
Copy link
Author

@alcohol I reverted changes for lookup the country names in ASCII.

So current PR is very simple, although I still plan to suggest ASCII lookup in another separate PR.

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.

None yet

2 participants