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

Add status_code to TravisError #38

Merged
merged 1 commit into from Oct 24, 2016
Merged

Conversation

adamchainz
Copy link
Contributor

I wanted to access it to filter 404's but the constructor was removing it from _contents so it wasn't accessible. This makes it a permanent attribute of the exception.

I wanted to access it to filter 404's but the constructor was removing it from _contents so it wasn't accessible. This makes it a permanent attribute of the exception.
@coveralls
Copy link

coveralls commented Oct 22, 2016

Coverage Status

Coverage decreased (-12.9%) to 87.131% when pulling d3200c3 on adamchainz:error into 3cb3be8 on menegazzo:master.

@menegazzo menegazzo merged commit d3ce7f3 into menegazzo:master Oct 24, 2016
@menegazzo
Copy link
Owner

Cool stuff!!! Thanks for this! 👍

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.

None yet

3 participants