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
Improve error handling for non existent releases #178
Conversation
Improve logging. Move code that gets the release commit hash inside try..catch.
No linked issues found. Please add the corresponding issues in the pull request description. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me but apart from error handling it doesn't seem to introduce any functional changes from the original implementation. I don't mind merging it, but did we figure out what was the cause of the issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll approve to avoid blocking. Needless to say it would be ideal to figure out the cause of the issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's repurpose this PR in terms of failing consistently when something unexpected happens:
- the draft release could not be created
- the draft release could not be found when trying to turn it into a published release
@simoneb I changed some small stuff and fixed the tests. After a quick review it should be ready to merge. It should now fail in both cases:
|
No description provided.