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: Save ccache cache from main only #28

Closed
wants to merge 1 commit into from

Conversation

auscompgeek
Copy link
Member

@auscompgeek auscompgeek commented Nov 7, 2023

Given the size of these caches, PR and tag builds will likely evict caches from the default branch after a few builds.

Got this idea from https://github.com/gradle/gradle-build-action#select-which-branches-should-write-to-the-cache

@virtuald
Copy link
Member

virtuald commented Nov 7, 2023

That feels like it sorta defeats the point? We already slow down builds on main because for the most part I don't care about them. I care more about tag builds (because I'm usually waiting for them so I can do the next step) and especially about PRs (because I want to see if changes succeed on windows).

It seems that it would be better to figure out why ccache isn't working on Windows when it claims to work. sccache just feels really inferior for this amount of cache artifacts.

@virtuald
Copy link
Member

virtuald commented Nov 7, 2023

I especially care about repeated PR builds, because typically I mess up and I'd like the second run to be faster.

@virtuald virtuald closed this Dec 7, 2023
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.

2 participants