Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix an error that didn't report the underlying error along with its context #38

Merged
merged 1 commit into from
Oct 27, 2022

Conversation

scr-oath
Copy link
Contributor

Context

@jithine and I recently spent time debugging an issue with using gitversion that just said error creating new tag and not what the error was - if we had known that the error was related to the tag already existing, finding the root cause would have been straightforward

Objective

Include the wrapped error in the error being reported when creating a new tag fails.

References

License

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@jithine jithine merged commit 8b9f7ac into screwdriver-cd:master Oct 27, 2022
@scr-oath scr-oath deleted the fix-error-log branch October 27, 2022 20:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants