-
-
Notifications
You must be signed in to change notification settings - Fork 410
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
Temporal: DateTime
and PlainDateTime
functionality
#3628
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3628 +/- ##
==========================================
- Coverage 47.43% 47.43% -0.01%
==========================================
Files 476 476
Lines 46288 46578 +290
==========================================
+ Hits 21959 22094 +135
- Misses 24329 24484 +155 ☔ View full report in Codecov by Sentry. |
Test262 conformance changes
Fixed tests (220):
Broken tests (14):
|
1de809d
to
9f59e52
Compare
9f59e52
to
846d49c
Compare
All of the broken tests were previously false positives from returning a |
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.
Nice work
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.
LGTM
This Pull Request is related to the ongoing work for #1804.
It changes the following:
boa_temporal
'sDateTime
.PlainDateTime
's accessor properties, accounting for the above changes.