-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Build: use lerna 2.x beta #3509
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
Conversation
Current coverage is 87.79%@@ master #3509 diff @@
==========================================
Files 194 194
Lines 9637 9515 -122
Methods 1101 1073 -28
Messages 0 0
Branches 2203 2163 -40
==========================================
- Hits 8472 8354 -118
+ Misses 1165 1161 -4
Partials 0 0
|
} | ||
}, | ||
"bootstrapConfig": { | ||
"ignore": "babel-runtime" |
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.
ignore babel-runtime
which should replace the rm -rf packages/*/node_modules/babel-runtime
above
3d10770
to
41fbde6
Compare
Great finally passed - still the question I brought up earlier though |
make build | ||
cd packages/babel-runtime; \ | ||
npm install; \ |
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.
Ignoring babel-runtime means it doesn't npm install
for babel-runtime which sucks
lerna changelog update
@@ -52,15 +52,14 @@ publish: | |||
rm -rf packages/*/lib | |||
BABEL_ENV=production make build-dist | |||
make test | |||
./node_modules/.bin/lerna publish | |||
./node_modules/.bin/lerna publish --only-explicit-updates |
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.
This emulates behavior in lerna 1.x
$ lerna updated --only-explicit-updates
Lerna v2.0.0-beta.23
Checking for updated packages...
- babel-code-frame
- babel-generator
- babel-plugin-transform-es2015-unicode-regex
- babel-plugin-transform-react-jsx-self
- babel-preset-es2016
- babel-preset-react
- babel-preset-stage-2
- babel-preset-stage-3
without the flag, it checks all dependencies and thus updates all packages
An interesting thing with the updates to
lerna updated
in 2.x.Maybe we do want an option to only update the packages that changed and not account for the sub-dependencies, especially since we are using
^
anyway?Adds repository field to babel-regenerator-runtime
changes package.json and in the end all packages depend on babel-regenetor-runtime because of babel-runtime. So simply changing that field inpackage.json
causes all to update which is kind of annoying atm.