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
Disallow dependency cycles #13442
Disallow dependency cycles #13442
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 0e454a1:
|
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/46760/ |
"@babel/types": "workspace:^7.13.16" | ||
}, | ||
"TODO": "The @babel/traverse dependency is only needed for the NodePath TS type. We can consider exporting it from @babel/core.", | ||
"devDependencies": { | ||
"@babel/traverse": "workspace:*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Traverse is only used for a type import:
babel/packages/babel-helper-hoist-variables/src/index.ts
Lines 1 to 3 in d3f4c22
import * as t from "@babel/types"; | |
import type { NodePath } from "@babel/traverse"; | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
command at nicolo-ribaudo/yarn-plugin-babel-release-tool@e0194c8
looks good if CI passes
It seems that this also fixes the failures on The "publish verdaccio" step was taking 5 hours (and then it was killed) because of an infinite loop in cycle detection, fixed by nicolo-ribaudo/yarn-plugin-babel-release-tool@e0194c8#diff-84e45058636f9e228c1d37fa19e6c943a505efa6e144a64ebb25df629233ff84R134-R135. |
We currently cannot publish because of a cycle. I'll push the cycle fix here, but first I want to see CI failing.