-
Notifications
You must be signed in to change notification settings - Fork 40
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
Push even if the build fails #33
Comments
Agreed, now that 0.3.7 is out I'll bring that back in. |
I just came around to report this too. Especially in an early phase of a repository with dependencies that builds for an hour or more (think ghcjs) this would be very useful! |
Planning to do this one in a couple of days. |
Thanks :) |
So does this still work in cachix-action-v4 or do I have to use cachix-action-v3? |
This is fixed on master, I'm testing if it also works when the build timeouts. |
Fixed via https://github.com/cachix/cachix-action/releases/tag/v6 WARNING: it's a breaking release, make sure to read changelog |
Until v4 the paths were pushed in parallel with the build, so the cache was populated even if the build would at the end fail. It seems that the current version updates the cache only if the build succeeds. This can be quite inconvenient for long builds.
What do you think about running the
Cachix: Pushing paths
group even ifInvoking nix-build
failed?The text was updated successfully, but these errors were encountered: