Skip to content

Move splitExportDeclaration to NodePath.prototype - #16645

Merged
nicolo-ribaudo merged 1 commit into
babel:feat-7.25.0/remove-cyclesfrom
nicolo-ribaudo:remove-cycles-3
Jul 17, 2024
Merged

nicolo-ribaudo merged 1 commit into
babel:feat-7.25.0/remove-cyclesfrom
nicolo-ribaudo:remove-cycles-3

Conversation

@nicolo-ribaudo

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

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

babel-bot commented Jul 15, 2024

Copy link
Copy Markdown
Collaborator

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