search

robotpy / mostrobotpy

Official Repository of python implementation of WPILib components
https://robotpy.github.io
Other
11 stars 14 forks source link

ci: Save ccache cache from main only #28

Closed auscompgeek closed 12 months ago

auscompgeek commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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