Skip to content

Conversation

@jonathanlloyd
Copy link
Contributor

No description provided.

@jonathanlloyd jonathanlloyd self-assigned this Jun 19, 2019
@codecov-io
Copy link

Codecov Report

Merging #23 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #23   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           4      4           
  Lines         479    479           
=====================================
  Hits          479    479
Impacted Files Coverage Δ
pusher_push_notifications/__init__.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3cac5af...7355a5e. Read the comment docs.

1 similar comment
@codecov-io
Copy link

codecov-io commented Jun 19, 2019

Codecov Report

Merging #23 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #23   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           4      4           
  Lines         479    479           
=====================================
  Hits          479    479
Impacted Files Coverage Δ
pusher_push_notifications/__init__.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3cac5af...7355a5e. Read the comment docs.

if 400 <= status_code < 500:
raise PusherValidationError(error_string)
elif 500 <= status_code < 600:
if 500 <= status_code < 600:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This < 600 is weird :D

@jonathanlloyd jonathanlloyd merged commit dec67aa into master Jun 19, 2019
@jonathanlloyd jonathanlloyd deleted the pin-linter-version branch June 19, 2019 13:10
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.

4 participants