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

chore(ci): deduplicate yarn cache by removing the set-up-job action #8504

Merged
merged 3 commits into from
Jun 3, 2023

Conversation

jtoar
Copy link
Contributor

@jtoar jtoar commented Jun 3, 2023

Follow up to #8449. With that PR in now, we have two yarn caches going at the same time that have the same content but different keys because most jobs still use the set-up-job action instead of the new set-up-yarn-cache action.

This does not affect how the test project is cached, it only deduplicates the caching for the framework's dependencies.

This avoids caching yarn dependencies under two different keys.
@jtoar jtoar added the release:chore This PR is a chore (means nothing for users) label Jun 3, 2023
@jtoar jtoar changed the title chore(ci): deduplicate yarn caches by removing the set-up-job action chore(ci): deduplicate yarn cache by removing the set-up-job action Jun 3, 2023
@replay-io
Copy link

replay-io bot commented Jun 3, 2023

@jtoar jtoar merged commit 3ff5c6b into main Jun 3, 2023
23 checks passed
@jtoar jtoar deleted the ds-ci/rm-set-up-job branch June 3, 2023 04:16
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Jun 3, 2023
@jtoar jtoar modified the milestones: next-release, v6.0.0 Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:chore This PR is a chore (means nothing for users)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant