-
Notifications
You must be signed in to change notification settings - Fork 6
Conversation
This a **major**, **breaking** change that revamps how this package extracts the CLDR data. In order to get better and more up-to-date CLDR data, this package now uses: `formatjs-extract-cldr-data` which uses `make-plural`. The data is now from CLDR v26.
CLA is valid! |
I've tested this locally with an updated |
locales : undefined, | ||
pluralRules : false, | ||
relativeFields: false, | ||
prelude : '', |
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.
unnecessary comma?
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 been doing this lately on purpose to avoid refactoring hazards. Since this is possible in ES5+ and that this package only runs in node, I figured it would be good to start using it. In fact trailing commas in function call expressions is on deck for ES7: https://github.com/tc39/ecma262
Seems it needs to update the README. |
@okuryu Yeah, I'll update the README for this PR, so it can be merged then published. |
🎱 |
This is in |
This a major, breaking change that revamps how this package extracts the CLDR data.
I'm currently finishing up testing these changes downstream, so this should not be merged until noted.In order to get better and more up-to-date CLDR data, this package now uses:
formatjs-extract-cldr-data
which usesmake-plural
. The data is now from CLDR v26.These changes supersede PR #1.