-
-
Notifications
You must be signed in to change notification settings - Fork 95
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
Case conversion with specific locale #72
Comments
I am experiencing same issue too. Just because of this error, I couldn't get Jest work without workarounds.
|
selimrecep
added a commit
to selimrecep/camelcase
that referenced
this issue
Jul 18, 2020
I have added optional 'locale' option to `options`
This was referenced Jul 18, 2020
I have created PR. I think that will fix it. |
Merged
@selimrecep Sorry, in parallel I've also submitted a PR. Any of them is fine :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Case conversion are done with
String.prototype.toLocaleLowerCase()
andString.prototype.toLocaleUpperCase()
without explicit locale parameter and this causes host dependent output.Example:
This causes unexpected results int hosts with non en-US environment.
I'd suggest to extend
options
parameter to have alocale
property to be passed to aforementioned methods.The text was updated successfully, but these errors were encountered: