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

Update test262 #17021

Merged
merged 1 commit into from
Dec 13, 2024
Merged

Update test262 #17021

merged 1 commit into from
Dec 13, 2024

Conversation

babel-bot
Copy link
Collaborator

@babel-bot
Copy link
Collaborator Author

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/58478

staging/sm/module/bug1693261-x.mjs(strict mode)
staging/sm/regress/regress-563221.js(default)
staging/sm/regress/regress-609617.js(default)
staging/sm/statements/for-of-iterator-close.js(default)
Copy link
Member

@nicolo-ribaudo nicolo-ribaudo Dec 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are all broken tests, and not Babel bugs.

Copy link
Member

@nicolo-ribaudo nicolo-ribaudo Dec 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

@nicolo-ribaudo nicolo-ribaudo Dec 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just asked to the maintainers. I think it's a bug in test262: all files except for those containing _FIXTURE are meant to be interpreted as JS tests, but I'll wait for them to answer.

@nicolo-ribaudo
Copy link
Member

CI failure is unrelated

@nicolo-ribaudo nicolo-ribaudo merged commit c7e9268 into main Dec 13, 2024
51 of 55 checks passed
@nicolo-ribaudo nicolo-ribaudo deleted the update-test262-parser branch December 13, 2024 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants