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] Fix yarn cache #9829

Merged
merged 12 commits into from
May 20, 2024
Merged

[CI] Fix yarn cache #9829

merged 12 commits into from
May 20, 2024

Conversation

fzaninotto
Copy link
Member

@fzaninotto fzaninotto commented May 3, 2024

Problem

The dependency installation never manages to store cache.

See e.g. https://github.com/marmelab/react-admin/actions/runs/8945903962/job/24575813530

saving npm cache failed with Error: Path Validation Error: Path(s) specified in the action for caching do(es) not exist, hence no cache is being saved., continuing...

Solution

Use GitHub's native node install step

Build time

Before After
5m 04s 4m 22s

Total build time

Before After
22m 08s 20m 48s

@fzaninotto fzaninotto merged commit 8e45399 into next May 20, 2024
13 checks passed
@fzaninotto fzaninotto deleted the yarn-cache branch May 20, 2024 09:58
@fzaninotto fzaninotto added this to the 5.0.0 milestone May 20, 2024
@fzaninotto fzaninotto mentioned this pull request May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFR Ready For Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants