Skip to content

Conversation

hzoo
Copy link
Member

@hzoo hzoo commented May 25, 2016

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 in package.json causes all to update which is kind of annoying atm.

@hzoo hzoo added the PR: Internal 🏠 A type of pull request used for our changelog categories label May 25, 2016
@codecov-io
Copy link

codecov-io commented Jun 14, 2016

Current coverage is 87.79%

Merging #3509 into master will decrease coverage by 0.11%

@@             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          

Powered by Codecov. Last updated by 3ee0618...76fb8ac

}
},
"bootstrapConfig": {
"ignore": "babel-runtime"
Copy link
Member Author

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

@hzoo hzoo force-pushed the lerna-2.x branch 3 times, most recently from 3d10770 to 41fbde6 Compare June 14, 2016 22:18
@hzoo
Copy link
Member Author

hzoo commented Jun 14, 2016

Great finally passed - still the question I brought up earlier though

make build
cd packages/babel-runtime; \
npm install; \
Copy link
Member Author

@hzoo hzoo Jun 15, 2016

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
Copy link
Member Author

@hzoo hzoo Jun 24, 2016

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

@hzoo hzoo added this to the Next Minor milestone Jun 24, 2016
@hzoo hzoo merged commit 22eb40d into master Jun 26, 2016
@hzoo hzoo deleted the lerna-2.x branch June 26, 2016 21:28
@hzoo hzoo modified the milestone: Next Minor Jul 16, 2016
@lock lock bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Oct 7, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 7, 2019
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 PR: Internal 🏠 A type of pull request used for our changelog categories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants