-
Notifications
You must be signed in to change notification settings - Fork 252
Add bun support as a package manager #475
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
@microsoft-github-policy-service agree |
|
Please check with us first if we want a feature, before investing work in a PR that we might not take. I haven't used If all that's needed is to replace |
I don't think so. Why shouldn't he be on the list? Who knows, he will choose. Moreover, yoeman is now execute
Does that make it harder? The usual choice is the same as between npm/yarn/pnpm... |
|
yeoman/environment@4.4.1 with Bun install released. Do I need to bump the version in dependencies? |
I suggest yo generator does not try to install dependencies by itself (it seems to be the only reason it asks you about the package manager, as far as I understand), and instead to make it prompt the user to install the dependencies by themselves. I believe generating the project skeleton/template is the only thing this generator should do. What do you think? |
Tjolk-072
left a comment
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.
Great work, thank you very much.
Description
This PR adds support for bun as a package manager.
Changes
Based on the existing tests, I've created additional tests to validate support for
bunwith all relevant templates.Checklist