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

Fix: error handling #449

Merged
merged 1 commit into from
May 7, 2024
Merged

Conversation

till
Copy link
Contributor

@till till commented May 6, 2024

  • refactored the big if/else to exit early
  • added error handling for different errors while creating a release

@till till mentioned this pull request May 6, 2024
- refactored the big `if/else` to exit early
- added error handling for different errors while creating a release
@softprops
Copy link
Owner

looks like an improvement to me. thanks!

@softprops softprops merged commit 0a76e42 into softprops:master May 7, 2024
1 check passed
@softprops
Copy link
Owner

heads up. these improvements have been incorporated into an updated v2 tag which now includes thanks. we can expand this list as we go. just a heads up, your changes have been incorporated into the v2 which now includes these changes

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.

3 participants