-
Notifications
You must be signed in to change notification settings - Fork 6
improvement(repo): Migrate from yarn to pnpm #45
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
base: master
Are you sure you want to change the base?
Conversation
}, | ||
"scripts": { | ||
"bootstrap": "lerna bootstrap", | ||
"build": "lerna run build --scope=react-isomorphic-data --stream", | ||
"bootstrap": "pnpm recursive 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.
pnpm install
will work the same way as pnpm recursive install
in a monorepo
"dev": "pnpm run dev --filter react-isomorphic-data", | ||
"example": "pnpm run start --filter client", | ||
"example:ssr": "pnpm run start --filter ssr", | ||
"deploy": "pnpm run test && HUSKY_SKIP_HOOKS=true lerna publish --conventional-commits --create-release github", |
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.
pnpm test
would also work
@@ -0,0 +1,3 @@ | |||
packages: | |||
- 'examples/*' | |||
- 'packages/*' |
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 will work for one level. If you have subdirs, use packages/**
@zkochan Whoa, thanks a lot for your review! Keep being awesome! |
The major problem now is the GH Action CI.
Waiting for either of these to be available: