-
-
Notifications
You must be signed in to change notification settings - Fork 156
feat: store caching #188
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
base: master
Are you sure you want to change the base?
feat: store caching #188
Conversation
|
Would also close #185 |
|
Thanks @oBusk, linked that issue too. |
|
If you would like to test this in your workflows, you can use the hash of the latest commit from this branch as a version: - name: Setup pnpm
# https://github.com/pnpm/action-setup/pull/188
uses: pnpm/action-setup@0bc9da32cd356746083d9e2b8848bad8b974c72c
with:
cache: true |
* no semicolons * no star imports * import order
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
pnpm can install Node runtime as a dependency, but
actions/setup-nodeis still needed to cache the pnpm store (as currently suggested in the readme). Alternative is to add something like this to your workflow:With this PR, the same can be accomplished simply by setting
cache: truein setup pnpm action:The implementation is based on actions/setup-node.
Closes #70, closes #80, closes #82, closes #152, closes #173.
Once stable, this could be enabled by default, although I think it would require a major version bump.