-
-
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
Build out Temporal's ZonedDateTime
and Now
#4068
Conversation
Test262 conformance changes
Fixed tests (185):
Broken tests (129):
|
Opened a PR to hopefully fix the compilation issues of this PR in boa-dev/temporal#125 |
…_os (#125) This PR is meant to address issues with boa-dev/boa#4068, which breaks when compiling for WASM targets. The changes were not made to the tests as I don't believe those will affect compilation. As a side note, we should probably be testing `temporal_rs` on more targets in CI if possible, especially with adding a FFI on the horizon.
Nice! Did you investigate the test regressions we have? |
I didn't dig too deep into test regression as I'm fairly certain glancing at the broken tests that they were more so false positives beforehand. But I can definitely double check. EDIT: huh, actually I'm going to dig into those more. The limit tests failing are interesting. |
Fixed the zonedDateTimeISO misspelled name. The other tests are probably somehow a regression in |
I'd also suggest updating the test262 commit, since I think some tests could have been adjusted since our last bump. |
So after looking at it, I'm fairly certain the remaining broken tests (prior to the test262 bump) are definitely something broken, but it's most likely a break in |
…en down from test262
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.
Looks good! We'll have to investigate the regexp tests, but since this didn't touch anything related, they must be updates of old tests.
This Pull Request is related to the ongoing work for #1804
It changes the following:
temporal_rs
tz_provider
field toContext
usingFsTzdbProvider
to provide initial time zone supportZonedDateTime
Now