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

Throw original to keep stack #22

Merged
merged 1 commit into from Oct 3, 2019

Conversation

@jmhardison
Copy link
Owner

commented Oct 3, 2019

Throwing original exception to keep appropriate stack.

Throwing original exception to keep appropriate stack.
@lgtm-com

This comment has been minimized.

Copy link

commented Oct 3, 2019

This pull request introduces 1 alert and fixes 1 when merging d6bcee7 into cf30ce9 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

fixed alerts:

  • 1 for Rethrowing exception variable
@jmhardison jmhardison merged commit 47071da into master Oct 3, 2019
1 of 3 checks passed
1 of 3 checks passed
continuous-integration/appveyor/branch Waiting for AppVeyor build to complete
Details
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
LGTM analysis: C# 1 new alert and 1 fixed alert
Details
@jmhardison jmhardison deleted the jmhardison-patch-1 branch Oct 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.