-
-
Notifications
You must be signed in to change notification settings - Fork 92
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
How does this compare to run: corepack enable
?
#105
Comments
The advantage is that this installs the "executable" version of pnpm, which is bundled with node.js using |
Is this really true? Isn't this a flag https://github.com/pnpm/action-setup/tree/v2#standalone ? |
Surely, an advantage over |
One downside is that corepack is still marked as experimental: https://nodejs.org/docs/latest-v20.x/api/corepack.html |
@segevfiner yes, it is turned off by default at the moment. There is nothing wrong with corepack. In our docs we actually recommend corepack for all CIs except Github actions: https://pnpm.io/continuous-integration. But there is nothing wrong with using corepack instead of this action. |
Edit: as of One downside of
Workaround 1Remove the - uses: pnpm/action-setup@v3
with:
- version: 'latest' Workaround 2Switch - - uses: pnpm/action-setup@v3
- with:
- version: 'latest'
+ - run: corepack enable Workaround 3Always keep |
@zkochan because of the (to avoid users running into this footgun, especially since Or, alternatively, if |
Doesn't it already have a feature to look at |
Indeed, I somehow missed this! As per the
However, this should probably error in the action when I'll create an issue |
Ok I ended up creating a PR for this instead - to throw an error if multiple versions of pnpm are specified: |
It seems that using core pack does not enable global installs?
https://github.com/graffle-js/graffle/actions/runs/11783197937/job/32819852527?pr=1259 |
Similar to #30 - I am wondering if there's an advantage to using this vs just running
corepack enable
, which as far as I understand, will instruct node itself to download and installpnpm
, and will respect thepackageManager
field in package.json.I'm doing this in my expect-type repo (permalink) and it seems to be working - is there a downside that you know of?
The text was updated successfully, but these errors were encountered: