chore: use npm7 workspaces instead of lerna bootstrap #3565
Conversation
22c7357
to
4669bc3
Compare
package.json
Outdated
"configure-examples:add-examples-lerna": "npx json -I -f ./lerna.json -e \"this.packages.push('examples/*'); this.packages = [...new Set(this.packages)]\"", | ||
"configure-examples:add-examples-npm": "npx json -I -f ./package.json -e \"this.workspaces.push('examples/*'); this.workspaces = [...new Set(this.workspaces)]\"", |
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.
if we define workspaces in the package.json we still need it in lerna.json ?
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.
For the time being yes - when v4.0.0 is stable we can probably get rid of it.
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.
Actually, I type that and lerna 4 is now tagged latest
instead of next
so maybe it's ready
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.
Though there's this: lerna/lerna#2841
268ce03
to
162cf34
Compare
It's much faster! npm@7 is required so people will need to upgrade if they are running node 14. Node 15 comes with npm@7 so all good there. ``` npm7 real 2m30.996s user 8m11.495s sys 0m48.380sit lerna real 4m19.727s user 4m36.371s sys 0m51.310s ```
162cf34
to
b0d686c
Compare
So close! Needs ipfs-examples/js-ipfs-examples#29 |
It's much faster! npm@7 is required so people will need to upgrade
if they are running node 14. Node 15 comes with npm@7 so all good
there.