-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add pnpm #7
Comments
Thank you for your suggestion! Contributions are also welcome if you'd like to help with this enhancement. I am having issues already with people having different versions of yarn causing crashes. |
@RobSchilderr If you add a pnpm branch I will make a PR with the changes 😄 |
Hey I added a pnpm branch! I think once it's done we can switch to pnpm for real and drop yarn. |
I have created a initial PR where I have tested the web app |
I hope to make time to check the native app sometime this week. |
Hi @OliverGeneser , We're encountering an issue with integrating PNPM as a package manager. The initial CLI installation relies on Supertokens, which currently doesn't support PNPM. Our standard process is to use npx create-supertokens-app@latest --manager=yarn for initializing new repositories. When attempting to use PNPM, we encounter the following error:
To address this, we have a few options:
For now I think option #3 is the best as I don't have the time to make our own CLI. If someone does, we can of course consider this. |
@RobSchilderr As far as I can see they only clone this repo and print a link to the repo. So I don't think the --manager has any effect when choosing the Capacitor template. So we could probably add a note in the README and just state that this Repo uses PNPM and they should just use npx create-supertokens-app@latest 😄 |
I merged the PNPM branch to main! |
Maybe pnpm would be a good addition to npm, yarn and bun 😃
The text was updated successfully, but these errors were encountered: