Uh oh!
There was an error while loading. Please reload this page.
Add documentation for actions/setup-node pnpm cache - #8389
Conversation
Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines. |
MaksimZhukov
left a comment
There was a problem hiding this comment.
Looks good to me, thank you! Nitpick: I think it make sense to mention the pnpm package manager in the Caching dependencies to speed up workflows guide.
6d99f0fjacobwgillespie
commented
Jul 20, 2021
Added! 👍 |
This comment has been minimized.
This comment has been minimized.
ramyaparimi
commented
Jul 21, 2021
@jacobwgillespie Thanks so much for opening a PR! I'll get this triaged for review ⚡ |
skedwards88
left a comment
There was a problem hiding this comment.
Thanks for adding this! I'm going to convert the disclaimer to a reusable, then I'll get this merged down for you.
Uh oh!
There was an error while loading. Please reload this page.
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues ⚡ |
Why:
The
actions/setup-nodeaction now supports pnpm caching as of v2.3.0, and @MaksimZhukovrequested a PR to update the docs for that addition.What's being changed:
I've added the workflow YAML example from https://github.com/actions/setup-node#caching-packages-dependencies for pnpm underneath the existing npm and yarn examples, matching the order from the action README.
Check off the following:
Writer impact (This section is for GitHub staff members only):