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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid numberingSystem tests #2455

Closed
longlho opened this issue Dec 24, 2019 · 2 comments
Closed

Invalid numberingSystem tests #2455

longlho opened this issue Dec 24, 2019 · 2 comments

Comments

@longlho
Copy link
Contributor

longlho commented Dec 24, 2019

Hi there,

While working on Unified NumerFormat polyfill I noticed #2383 which might not be valid as numberingSystem is not retrieved anywhere in current spec and proposed unified numberformat. The closest call was the iteration over options in https://tc39.es/ecma402/#sec-resolvelocale but that's not enough to trigger a getter. Should those tests be modified?

cc @FrankYFTang @sffc

Thanks,

LH

@sffc
Copy link
Contributor

sffc commented Jan 8, 2020

The test is for tc39/ecma402#175, which adds those options and how to read them in Intl objects.

@longlho
Copy link
Contributor Author

longlho commented Jan 8, 2020

Gotcha thanks!

@longlho longlho closed this as completed Jan 8, 2020
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

No branches or pull requests

2 participants