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

Defer to built-in action caching #1938

Merged
merged 7 commits into from
Aug 21, 2024
Merged

Defer to built-in action caching #1938

merged 7 commits into from
Aug 21, 2024

Conversation

boardfish
Copy link
Collaborator

@boardfish boardfish commented Dec 17, 2023

Closes #1937

Instead of using actions/cache directly, defer to the setup-ruby and setup-node actions' native support, which uses this under the hood.

@Spone
Copy link
Collaborator

Spone commented Dec 17, 2023

is there a reason we run on 2.7 in these?

I'd say it's just legacy.

@joelhawksley
Copy link
Member

@boardfish any interest in bringing this PR up to date?

@boardfish
Copy link
Collaborator Author

Yep, I'll take a look at this soon!

@boardfish boardfish force-pushed the built-in-action-caching branch 2 times, most recently from 45ddf5b to b1fc788 Compare August 17, 2024 09:21
@boardfish boardfish marked this pull request as ready for review August 17, 2024 09:24
@boardfish
Copy link
Collaborator Author

This is good to go – the failures on Rails main are unrelated.

@joelhawksley
Copy link
Member

@boardfish I think we might have one more real failure here ❤️

@boardfish boardfish force-pushed the built-in-action-caching branch 2 times, most recently from 4ee00c9 to d87e000 Compare August 21, 2024 08:30
Uses the Rails version specified in Gemfile.lock.
@boardfish
Copy link
Collaborator Author

That's now fixed 👍

@boardfish boardfish enabled auto-merge (squash) August 21, 2024 09:56
@boardfish boardfish merged commit d090fb9 into main Aug 21, 2024
19 checks passed
@boardfish boardfish deleted the built-in-action-caching branch August 21, 2024 14:49
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.

Use setup-ruby's native bundle caching in our workflows
3 participants