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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(workflow): add cache to workflows using actions/setup-node #404

Closed
wants to merge 5 commits into from
Closed

ci(workflow): add cache to workflows using actions/setup-node #404

wants to merge 5 commits into from

Conversation

oscard0m
Copy link

@oscard0m oscard0m commented Jul 5, 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! 💪🏾

Copy link
Collaborator

@Turbo87 Turbo87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

basically the same comments as on mainmatter/ember-test-selectors#761 😉

.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
@oscard0m oscard0m requested a review from Turbo87 July 5, 2021 10:24
@Turbo87
Copy link
Collaborator

Turbo87 commented Feb 11, 2022

whoops, sorry, missed this PR. it's implemented now via #424 😅

@Turbo87 Turbo87 closed this Feb 11, 2022
@oscard0m
Copy link
Author

whoops, sorry, missed this PR. it's implemented now via #424 😅

No problem! 😎

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.

2 participants