Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(workflow): add 'npm' cache for actions/setup-node in .github/workflows #179

Merged
merged 1 commit into from Dec 20, 2021
Merged

ci(workflow): add 'npm' cache for actions/setup-node in .github/workflows #179

merged 1 commit into from Dec 20, 2021

Conversation

oscard0m
Copy link
Contributor

@oscard0m oscard0m commented Dec 20, 2021

Description

Add cache to workflows using actions/setup-node

Context

setup-node GitHub Action just released a new option to add cache to steps using it.

You can find the details here: https://github.blog/changelog/2021-07-02-github-actions-setup-node-now-supports-dependency-caching/


馃 This PR has been generated automatically by this octoherd script, feel free to run it in your GitHub user/org repositories! 馃挭馃従

@oscard0m
Copy link
Contributor Author

oscard0m commented Dec 20, 2021

@brcrista regarding the exact earnings I'm not sure how to measure it in this repository since I don't have control to run the workflow and compare.

Here you can see a table with the expected improvements:
actions/setup-node#271

@brcrista
Copy link
Contributor

brcrista commented Dec 20, 2021

@oscard0m you should be able to run workflows in your fork. But not a big deal, caching seems sensible here and is generally what we would recommend.

@brcrista brcrista merged commit 424fc82 into actions:main Dec 20, 2021
28 checks passed
@oscard0m oscard0m deleted the add-cache-to-node-workflows branch Dec 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants