-
Notifications
You must be signed in to change notification settings - Fork 320
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
"Install tools" always runs, even when there was a cache hit #3543
Comments
Would using something like https://github.com/Swatinem/rust-cache help? It’s prevented repeated downloading and compiling even when everything’s using the latest version of Node. |
We shouldn't need another action, we just have to get the github native action to behave... |
Ah, in https://github.com/rust-lang/miri/actions it did work properly. Maybe I was just checking the wrong actions, or it did take a bit to propagate the updated cache from #3537. Sorry for the noise! |
No worries! |
Looking at https://github.com/rust-lang/miri/actions, all the recent jobs are running this step, even though that should only happen when the cache is cold:
This is likely caused by #3519; things worked fine before that PR. There's a reason why I'd prefer if Github didn't subject us to updates like that all the time... they always tend to take time away from actual development...
Cc @hamirmahal
The text was updated successfully, but these errors were encountered: