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

[close #109] Allow retrying failed release #119

Merged
merged 2 commits into from
Aug 20, 2020

Conversation

schneems
Copy link
Contributor

When a deploy succeeds but the release fails the git remote retains the contents of the git push. This means if you immediately try to push again that you'll get a Everything up-to-date from the remote and Heroku will not try to re-build your project.

We don't want that behavior, we want to retry on a failed release incase it's intermittent. This PR adds an empty commit when a failed release is detected.

@schneems schneems force-pushed the schneems/retry-failed-release branch 3 times, most recently from b659f26 to e3ace08 Compare August 20, 2020 16:02
When a deploy succeeds but the release fails the git remote retains the contents of the `git push`. This means if you immediately try to push again that you'll get a `Everything up-to-date` from the remote and Heroku will not try to re-build your project.

We don't want that behavior, we want to retry on a failed release incase it's intermittent. This PR adds an empty commit when a failed release is detected.
@schneems schneems force-pushed the schneems/retry-failed-release branch from e3ace08 to 46bd67b Compare August 20, 2020 16:07
When `allow_failure` is set to true then it over-writes the value set by `retries`, this test asserts that to be true.
@schneems schneems merged commit 039f8bf into main Aug 20, 2020
@schneems schneems deleted the schneems/retry-failed-release branch August 20, 2020 17:04
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.

1 participant