-
Notifications
You must be signed in to change notification settings - Fork 158
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
Editorial: Recommend "remapping" for dates exceeding era year bounds #2986
Conversation
Ref tc39#2865
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2986 +/- ##
==========================================
+ Coverage 96.35% 96.40% +0.04%
==========================================
Files 21 21
Lines 10519 10575 +56
Branches 1803 1808 +5
==========================================
+ Hits 10136 10195 +59
+ Misses 325 322 -3
Partials 58 58 ☔ View full report in Codecov by Sentry. |
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.
Thanks, this all looks good to me.
I guess we have no test262 coverage for this. Are you interested in adding some?
Co-authored-by: Philip Chimento <philip.chimento@gmail.com>
I don't think I'll have availability for that until after the upcoming meeting, so feel free to jump ahead if you can. |
…monkey-reviewers,dminor Changes from: tc39/proposal-temporal#2986 Differential Revision: https://phabricator.services.mozilla.com/D231122
…monkey-reviewers,dminor Changes from: tc39/proposal-temporal#2986 Differential Revision: https://phabricator.services.mozilla.com/D231122
Fixes #2865