-
Notifications
You must be signed in to change notification settings - Fork 7.4k
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
feat: Switch from npm to pnpm #4429
Conversation
0874cf9
to
91198b1
Compare
91198b1
to
25f55dd
Compare
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.
Finally, we got here! Thanks for taking the effort! I left some comments and would be happy if someone else would also have a look.
dc8bdd3
to
3496eac
Compare
60659c2
to
93fec0e
Compare
…ndency requirement
…de_modules` folder this is needed to keep n8n backward compatible for the cloud
93fec0e
to
1ab75ad
Compare
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.
Tested via pnpm: start
, build
, dev
, clean
, lint
, lintfix
, format
e9d7f85
to
f121198
Compare
f121198
to
9ee47f2
Compare
9ee47f2
to
4d9b917
Compare
Got released with |
I've spent an hour to find out a real reason why IMHO - the Internet should be Opened for all individuals whenever from his residence, region and personaliy. So I recommend to not use pnpm package manager in n8n and switch to other well working alternatives like yarn or bit. |
@borisbsu You don't need pnpm to install n8n. pnpm is only needed for our development setup. We tried yarn first, as many of us have prior experience with yarn, but yarn doesn't have an equivalent of However we do agree with you that the Internet should be open to app, and that's why the we provide multiple ways to install n8n, none of which require the end-user to even know what pnpm is. |
No description provided.