Conversation
liuxingbaoyu
commented
Feb 22, 2026
| Q | A |
|---|---|
| Fixed Issues? | Fixes #17821 |
| Patch: Bug Fix? | |
| Major: Breaking Change? | |
| Minor: New Feature? | |
| Tests Added + Pass? | Yes |
| Documentation PR Link | |
| Any Dependency Changes? | |
| License | MIT |
|
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/61102 |
|
commit: |
7ccec94 to
09fb285
Compare
| pack: "yarn", | ||
| }); | ||
| for (const message of result.messages) { | ||
| if (message.type === "error") { |
There was a problem hiding this comment.
Can we output error to stderr if there is an error message?
7e6b376 to
3f2e23a
Compare
|
CI failures are expected. we need to remove the failed CIs or retain the compatible exports. |
a275a42 to
c14eac8
Compare
nicolo-ribaudo
left a comment
There was a problem hiding this comment.
Could you split this PR in two, so that we can enable linting after the release, and get it to pass?
|
This check is currently checking the local repository. The CI failed because I removed https://github.com/babel/babel/pull/17822/changes#diff-6a9f4fa54adb5d9a89a76a35a55ccbb649bec45ee9a74b91d39d9d1d52415eacL127-L128. |
|
Oh I see, so after merging it should pass? |
|
https://github.com/babel/babel/actions/runs/22495530191/job/65168898709?pr=17822 |
|
Could you take a look at the CI failures? |
|
Do you want to remove these failing CIs (nodejs < v17) or restore compatibility? |
|
I think |
|
I will revert that change. |
The current Babel-runtime 7 sets |
|
I'll merge this and prepare a PR for that. |