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 #6202

Closed
wants to merge 1 commit into from
Closed

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

wants to merge 1 commit into from

Conversation

oscard0m
Copy link

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! 💪🏾

@davorpa davorpa added 👥 discussion This Repo is guided by its community! Let's talk! New Feature New feature / enhancement / translation... labels Oct 12, 2021
@davorpa
Copy link
Member

davorpa commented Oct 12, 2021

Locked by #5564, #5521

This is really needed since project hasn't a package.json Node.JS config file?


Also updates action to v2

@davorpa davorpa added the conflicts Conflict(s) need to be resolved label Nov 7, 2021
@davorpa
Copy link
Member

davorpa commented Nov 7, 2021

⚠️ WARNING


Source fork/branch where this Pull Request (PR) was opened has been deleted.

Is this contribution valuable? Has unresolvable conflicts or no write permisions and, is necessary to carry it forward?

Good news 😉! As a detached way, it could be recovered from upstream by anybody using repo HEAD refs.

Use the following GIT commands to preserve all related commits history:

git fetch upstream pull/6202/head:pr/oscard0m/6202-add-cache-to-node-workflows
git switch pr/oscard0m/6202-add-cache-to-node-workflows
git push -u origin pr/oscard0m/6202-add-cache-to-node-workflows

/labelthis: detached

@oscard0m
Copy link
Author

oscard0m commented Nov 7, 2021

Locked by #5564, #5521

This is really needed since project hasn't a package.json Node.JS config file?


Also updates action to v2

If it does not use any package-lock there is no point of using this. Good point

@oscard0m oscard0m closed this Nov 7, 2021
@davorpa
Copy link
Member

davorpa commented Nov 7, 2021

Thanks for the response, share knowledge and your effort for trying to improve the project 😉

@davorpa davorpa added the 🤖 automation Automated tasks done by workflows or bots label Mar 5, 2022
@EbookFoundation EbookFoundation locked as resolved and limited conversation to collaborators Mar 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🤖 automation Automated tasks done by workflows or bots 👥 discussion This Repo is guided by its community! Let's talk! conflicts Conflict(s) need to be resolved New Feature New feature / enhancement / translation...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants