-
-
Notifications
You must be signed in to change notification settings - Fork 165
Add --yarn option to use yarn instead of npm #293
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: main
Are you sure you want to change the base?
Conversation
I don't know how to test this without code being published in npm registry. |
Probably more changes needed, as this did not have any effect at all |
Perhaps yes |
Moreover, I do not see similar testing for npm either. But I noticed that the readme template generator should also be changed here |
PR for readme sub-generator: let's get that resolved and merged first. |
Rebased after #294 merge I don't use this anymore, so either merge or close. |
👋 Thanks for the merge->rebase! Coming over from yeoman/yeoman#1779, we're still ramping up on PR infrastructure and reviews. Lots of things are broken and need to be overhauled. But we're hopeful we'll get to this Soon ™️! |
Just realized that corepack integration can be added. https://nodejs.org/api/packages.html#packagemanager. |
Fixes #289
use
yarnInstall
if--yarn option
is used: