nextjs-native-starter icon indicating copy to clipboard operation
nextjs-native-starter copied to clipboard

Add pnpm

Open OliverGeneser opened this issue 2 years ago • 8 comments

Maybe pnpm would be a good addition to npm, yarn and bun :smiley:

OliverGeneser avatar Nov 05 '23 18:11 OliverGeneser

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 avatar Nov 10 '23 06:11 RobSchilderr

@RobSchilderr If you add a pnpm branch I will make a PR with the changes :smile:

OliverGeneser avatar Nov 10 '23 19:11 OliverGeneser

@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.

RobSchilderr avatar Nov 13 '23 11:11 RobSchilderr

@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

OliverGeneser avatar Nov 13 '23 16:11 OliverGeneser

@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.

RobSchilderr avatar Nov 20 '23 10:11 RobSchilderr

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:

Error: Invalid package manager provided, valid values:
    - npm
    - yarn
    - bun

Error: If you think this is an issue with the tool, please report this at https://github.com/supertokens/create-supertokens-app/issues

To address this, we have a few options:

  1. Develop our own CLI that supports multiple package managers.
  2. Request Supertokens to add PNPM support.
  3. Maintain the current PNPM branch, allowing interested users to clone from there and clearly state this in the README.md.

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 avatar Dec 03 '23 09:12 RobSchilderr

@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 :smile:

OliverGeneser avatar Dec 03 '23 10:12 OliverGeneser

@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 :smile:

I merged the PNPM branch to main!

RobSchilderr avatar Dec 03 '23 22:12 RobSchilderr