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
Update @babel/*
deps
#11899
Update @babel/*
deps
#11899
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 c3af30a:
|
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/26775/ |
a41d317
to
c3af30a
Compare
@@ -105,8 +105,6 @@ module.exports = function (api) { | |||
// TODO: Use @babel/preset-flow when | |||
// https://github.com/babel/babel/issues/7233 is fixed | |||
"@babel/plugin-transform-flow-strip-types", | |||
["@babel/proposal-class-properties", { loose: true }], |
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.
nice!
@@ -252,8 +252,6 @@ endif | |||
bootstrap-only: lerna-bootstrap | |||
|
|||
yarn-install: clean-all | |||
# Install dependencies in individual packages so that we can link them at the top level. | |||
for package in eslint/*/; do yarn --ignore-engines --cwd $$package; done |
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.
Don't we still need this until Babel 8?
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.
Running yarn install in sub folders incurs changes to the yarn lock files after lerna version, lerna will abort publishing with uncommitted changes. See also the PR descriptions.
@babel/*
is updatedIn this PR we also removes extra install step for
@babel/eslint-*
.We have run into issues during publish: Let's say we want to publish 7.12. After
lerna version
, runningyarn install
will request unpublished 7.12@babel/*
deps from eslint packages, although we can choose the published version 7.11 resolved as 7.12, it will incur changes onyarn.lock
and thus the publish will fail because uncommitted yarn.lock file. See https://app.circleci.com/pipelines/github/babel/babel/3558/workflows/05ae840c-67c4-4312-9776-51f33f445325/jobs/26769 for the reproducing publish error.I have replaced two eslint packages from
link:
to published version. Lerna should handle them properly. Two private eslint packages are reserved as link. It happens to work because they don't have dependencies.Also simplified babel config by removing included plugins.