Skip to content

Add test262 for Babel 8#17338

Merged
nicolo-ribaudo merged 1 commit into
babel:mainfrom
liuxingbaoyu:test262-babel8
May 29, 2025
Merged

Add test262 for Babel 8#17338
nicolo-ribaudo merged 1 commit into
babel:mainfrom
liuxingbaoyu:test262-babel8

Conversation

@liuxingbaoyu

Copy link
Copy Markdown
Member
Q                       A
Fixed Issues? Fixes #1, Fixes #2
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

@babel-bot

Copy link
Copy Markdown
Collaborator

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

@liuxingbaoyu liuxingbaoyu changed the title Add test262 for babel8 Add test262 for Babel 8 May 29, 2025
@nicolo-ribaudo nicolo-ribaudo merged commit d1a7dcb into babel:main May 29, 2025
56 of 58 checks passed
@nicolo-ribaudo

Copy link
Copy Markdown
Member

@liuxingbaoyu I merged this so we can start building the baseline results for Babel 8, but could change it to run in parallel chunks like for Babel 7? The job takes almost one hour otherwise.

@liuxingbaoyu

Copy link
Copy Markdown
Member Author

test262 has caching for Babel 8, let's see how long it takes after caching. :)

@nicolo-ribaudo

Copy link
Copy Markdown
Member

I'm very confused by caching, I thought it was to cache when the tests change and the Babel code doesn't? 😅

Because if the Babel code changes, how do we know for which tests the cache needs to be invalidated?

@liuxingbaoyu

Copy link
Copy Markdown
Member Author

Before starting node v0.10 processes, we check the Babel-transformed code and all untransformed dependencies.
Whether Babel changes or the tests change, the cache will detect the changes immediately!

@github-actions github-actions Bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Aug 29, 2025
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 29, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants