Skip to content
This repository has been archived by the owner on Feb 23, 2024. It is now read-only.

Run npm install once and cache node_modules in E2E tests in CI #5689

Closed
alexflorisca opened this issue Feb 1, 2022 · 1 comment · Fixed by #6404
Closed

Run npm install once and cache node_modules in E2E tests in CI #5689

alexflorisca opened this issue Feb 1, 2022 · 1 comment · Fixed by #6404
Assignees
@alexflorisca
Labels
tools Used for work on build or release tools. type: cooldown Things that are queued for a cooldown period (assists with planning). type: enhancement The issue is a request for an enhancement.

Comments

@alexflorisca
Copy link
Member

Is your feature request related to a problem? Please describe.

It would speed up running the E2E tests in CI if we had a setup step that cached the node_modules for all the other steps in the workflow. There was already a Setup step that was removed in #5678 because it wasn't actually caching the node_modules. It would be good to get this working properly.
@alexflorisca alexflorisca added type: enhancement The issue is a request for an enhancement. tools Used for work on build or release tools. labels Feb 1, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Apr 3, 2022

This issue has been marked as stale because it has not seen any activity within the past 60 days. Our team uses this tool to help surface issues for review. If you are the author of the issue there's no need to comment as it will be looked at.

Internal: After 10 days with no activity this issue will be automatically be closed.

@github-actions github-actions bot added the status: stale Stale issues and PRs have had no updates for 60 days. label Apr 3, 2022
@alexflorisca alexflorisca removed the status: stale Stale issues and PRs have had no updates for 60 days. label Apr 4, 2022
@ralucaStan ralucaStan added the type: cooldown Things that are queued for a cooldown period (assists with planning). label May 4, 2022
@wavvves wavvves assigned wavvves and alexflorisca and unassigned wavvves May 5, 2022
@alexflorisca alexflorisca mentioned this issue May 11, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@alexflorisca alexflorisca

Labels
tools Used for work on build or release tools. type: cooldown Things that are queued for a cooldown period (assists with planning). type: enhancement The issue is a request for an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Cache node_modules between workflows woocommerce/woocommerce-blocks
3 participants
@ralucaStan @alexflorisca @wavvves