-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 #2279
Comments
Hi @autumnblazey, I am closing this issue. Please, feel free to create a new one in case if any other help is needed. |
edit: fixed! I believe FYI to anyone who finds this, the action hasn't had any updates since after the add-path command was deprecated and disabled, so a workaround (for now at least) is to use the setup node action (it doesnt work without using this, i've tried), then running steps:
- name: Setup Node.js environment
uses: actions/setup-node@v2.1.4
with:
node-version: "15"
- name: install pnpm
run: npm i -g pnpm |
Once actions/setup-node#482 is merged, you can use corepack instead |
that would be nice, thanks for the heads up! |
Tool information
Area for Triage:
JavaScript and Node.js
Question, Bug, or Feature?:
feature
Virtual environments affected
Can this tool be installed during the build?
yes
pnpm install
for example)Tool installation time in runtime
~3-5 seconds
Are you willing to submit a PR?
not completely sure how the runner works so I'll have to take a look around in the repo, but I'm willing to PR
The text was updated successfully, but these errors were encountered: