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

Leave the initial exception as final exception #1300

Closed
wants to merge 1 commit into from
Closed

Leave the initial exception as final exception #1300

wants to merge 1 commit into from

Conversation

lmazuel
Copy link
Member

@lmazuel lmazuel commented Jul 21, 2016

Do not override the initial exception by a useless message.

Fix Azure/azure-sdk-for-python#592

@annatisch for review please :)

@azurecla
Copy link

Hi @lmazuel, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@annatisch
Copy link
Member

@lmazuel - This will change some of the LRO tests which check for specific messages. I'll try to get this sorted for you.

@lmazuel lmazuel added the Python label Jul 21, 2016
@annatisch annatisch mentioned this pull request Sep 1, 2016
@annatisch
Copy link
Member

Resolved in PR #1392

@annatisch annatisch closed this Sep 2, 2016
@lmazuel lmazuel deleted the better_exception branch February 14, 2017 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants