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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a Special Case for Translating Choices in en_US_POSIX #42068

Merged

Conversation

chrisguitarguy
Copy link
Contributor

@chrisguitarguy chrisguitarguy commented Jul 12, 2021

Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #42036
License MIT
Doc PR n/a

This adds a special case similar to pt_BR for handling en_US_POSIX
which is set as the default locale on some operating systems.

Two tests here might be over the top, not sure.

Not over the top! One of them caught the same issue in the translation component itself 馃帀

See symfony#42036

This adds a special case similar to `pt_BR` for handling `en_US_POSIX`
which is set as the default locale on some operating systems (Alpine
Linux, for instance).
@nicolas-grekas
Copy link
Member

Thank you @chrisguitarguy.

@nicolas-grekas nicolas-grekas merged commit 5e80445 into symfony:4.4 Jul 13, 2021
@chrisguitarguy chrisguitarguy deleted the chrisguitarguy/issue-42036 branch July 13, 2021 11:40
This was referenced Jul 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants