Skip to content
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

[DevExperience] Replace yarn with npm #635

Open
huchenlei opened this issue Jan 16, 2025 · 2 comments
Open

[DevExperience] Replace yarn with npm #635

huchenlei opened this issue Jan 16, 2025 · 2 comments
Assignees

Comments

@huchenlei
Copy link
Member

huchenlei commented Jan 16, 2025

https://github.com/Comfy-Org/ComfyUI_frontend and https://github.com/Comfy-Org/litegraph.js are all using npm. Switch to npm in this project would better align the toolchain consistency, and improve onboarding experience for someone new to the team.

┆Issue is synchronized with this Notion page by Unito

@huchenlei huchenlei self-assigned this Jan 16, 2025
@webfiltered
Copy link
Contributor

I know one of our big frameworks very strongly recommends yarn - electron-builder, possibly.

@huchenlei
Copy link
Member Author

I know one of our big frameworks very strongly recommends yarn - electron-builder, possibly.

Following the discussion electron-userland/electron-builder#1147 (comment), yarn was recommended at that time because it has autoclean command, but it is actually not used in our setup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants