Skip to content

Fixes NumericFormatter failure on systems with other decimal separator#443

Merged
testforstephen merged 1 commit intomicrosoft:mainfrom
crissNb:main
Sep 13, 2022
Merged

Fixes NumericFormatter failure on systems with other decimal separator#443
testforstephen merged 1 commit intomicrosoft:mainfrom
crissNb:main

Conversation

@crissNb
Copy link
Copy Markdown
Contributor

@crissNb crissNb commented Sep 11, 2022

In certain regions (e.g. Brazil, Germany, etc.), a comma is used instead of a dot as a decimal separator by default.
This change will set the default locale to US (so a dot is used as a decimal separator) before performing numeric formatter tests.

This PR fixes #337. Solution already addressed by @siqueira-gustavo (full credit goes to him), but no changes have been made.

@testforstephen testforstephen merged commit f9592bf into microsoft:main Sep 13, 2022
@testforstephen
Copy link
Copy Markdown
Contributor

@crissNb472 thanks for contribution.

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.

NumericFormatter should be able to format double correctly. expected:<111[.]000000> but was:<111[,]000000>

2 participants