Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Add yarn script helper for linking all packages #476

Merged
merged 1 commit into from
Nov 21, 2017
Merged

Add yarn script helper for linking all packages #476

merged 1 commit into from
Nov 21, 2017

Conversation

eliperelman
Copy link
Member

No description provided.

@eliperelman eliperelman requested a review from a team November 21, 2017 17:39
package.json Outdated
@@ -14,6 +14,8 @@
"packages/*"
],
"scripts": {
"bootstrap": "lerna bootstrap",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The bootstrap step was removed in #372 since my understanding was that lerna's bootstrap itself just runs yarn install, which will have already occurred. Does it do something else that I've missed?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's possible they do the same thing, yarn and yarn lerna bootstrap, but I've been using both. I'll try it out and see what happens.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LOL you're right. I just nuked everything with yarn lerna clean, then yarn and yarn test and everything passes. I'll revise.

@eliperelman eliperelman changed the title Add helpers for bootstrapping with lerna and linking all packages Add yarn script helper for linking all packages Nov 21, 2017
@eliperelman eliperelman merged commit f934263 into neutrinojs:master Nov 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

2 participants