-
Notifications
You must be signed in to change notification settings - Fork 517
Migrate to pnpm #2276
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
Migrate to pnpm #2276
Conversation
3703c73
to
be24af3
Compare
Thanks for this, @jrmajor, looks great! I've updated it to reflect the latest changes and added some more scripts. Just wondering if you thought there was anything left to do since you marked it as draft? |
Thank you @pascalbaljet! IIRC, i just wanted to do some more testing to make sure that this doesn't break anything. Let me do a quick self-review to see if there are any todos left. |
@pascalbaljet Ready for review. Are the |
Just a shortcut to spin up the test server so we can easily tinker with it. |
I renamed the |
I think that usually |
Why not use bun? It is faster and more modern. |
I've put some more work into this the last couple of days. I think this PR makes working on Inertia much easier:
Also, I've rewritten the contributing guide so it should be easier for people to get started. |
I checked the built assets and they're exactly the same before and after this PR. The only exception is the Svelte package, where the only change is the absence of an unused import. |
Replaces #1875.