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.

Sign up for GitHub

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

Jump to bottom

ECMA-402 PR coverage: Add calendar and numberingSystem options #2331

Closed
leobalter opened this issue Sep 5, 2019 · 3 comments
Closed

ECMA-402 PR coverage: Add calendar and numberingSystem options #2331

leobalter opened this issue Sep 5, 2019 · 3 comments
Labels
coverage ECMA-402 ECMA-402 related PRs

Comments

@leobalter
Copy link
Member

leobalter commented Sep 5, 2019 •
edited
Loading

Ref: tc39/ecma402#175

I've might already have tests for this, see tc39/ecma402#175 (comment)

Close it if coverage is confirmed.
@caiolima caiolima added the ECMA-402 ECMA-402 related PRs label Dec 12, 2019
@caiolima
Copy link
Contributor

We are missing coverage on invalid options throwing RangeError.

@caiolima
Copy link
Contributor

BTW, I'm going to work on that now.

@caiolima caiolima mentioned this issue Jan 23, 2020
Merged
@leobalter
Copy link
Member Author

Closing after #2480

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
coverage ECMA-402 ECMA-402 related PRs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@leobalter @caiolima