-
Notifications
You must be signed in to change notification settings - Fork 730
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 simple polyfill for Intl.ListFormat #1585
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a test for this.
You can see how the other polyfill tests work, for example test/info/localeWeek.test.js
.
6edaf37
to
09a9314
Compare
09a9314
to
23a368a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added the tests
|
||
Helpers.withoutLF("Duration#toHuman falls back to commas", () => { | ||
expect(dur().reconfigure({ locale: "ja" }).toHuman()).toEqual( | ||
"1 年, 2 か月, 1 週間, 3 日, 4 時間, 5 分, 6 秒, 7 ミリ秒" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I picked Japanese since it uses a different comma style
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@diesieben07 Hey sorry for the tag, just checking if u had a look at this
Fixes #1189