Skip to content

Conversation

nicolo-ribaudo
Copy link
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

This follows the same path as #16577 and #16644, trying to remove type circular deps so that we can publish .d.ts definitions.

@babel/helper-helper-split-export-declaration works on NodePaths and is a dependency of @babel/traverse, so they will be always installed together. I moved it to NodePath.prototype, in the "conversions" section together with utils like arrowFunctionToExpression.

@nicolo-ribaudo nicolo-ribaudo added PR: New Feature 🚀 A type of pull request used for our changelog categories pkg: traverse labels Jul 15, 2024
@nicolo-ribaudo nicolo-ribaudo added this to the v7.25.0 milestone Jul 15, 2024
@babel-bot
Copy link
Collaborator

babel-bot commented Jul 15, 2024

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

@nicolo-ribaudo nicolo-ribaudo added the PR: Ready to be Merged A pull request with already two approvals, but waiting for the next minor release label Jul 16, 2024
@nicolo-ribaudo nicolo-ribaudo changed the base branch from main to feat-7.25.0/remove-cycles July 17, 2024 12:06
@nicolo-ribaudo nicolo-ribaudo merged commit 054be45 into babel:feat-7.25.0/remove-cycles Jul 17, 2024
@nicolo-ribaudo nicolo-ribaudo deleted the remove-cycles-3 branch July 17, 2024 12:14
@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 Oct 19, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 19, 2024
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 pkg: traverse PR: New Feature 🚀 A type of pull request used for our changelog categories PR: Ready to be Merged A pull request with already two approvals, but waiting for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants