Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

feat: remove unused udp wake capability #1001

Merged
merged 1 commit into from
Sep 7, 2017
Merged

feat: remove unused udp wake capability #1001

merged 1 commit into from
Sep 7, 2017

Conversation

bbangert
Copy link
Member

@bbangert bbangert commented Sep 5, 2017

Closes #886

@codecov-io
Copy link

codecov-io commented Sep 5, 2017

Codecov Report

Merging #1001 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1001      +/-   ##
==========================================
- Coverage   99.78%   99.78%   -0.01%     
==========================================
  Files          57       57              
  Lines        9656     9593      -63     
==========================================
- Hits         9635     9572      -63     
  Misses         21       21
Impacted Files Coverage Δ
autopush/config.py 100% <ø> (ø) ⬆️
autopush/router/webpush.py 100% <ø> (ø) ⬆️
autopush/tests/test_websocket.py 100% <ø> (ø) ⬆️
autopush/tests/test_main.py 100% <ø> (ø) ⬆️
autopush/websocket.py 100% <ø> (ø) ⬆️
autopush/tests/test_router.py 100% <ø> (ø) ⬆️
autopush/main_argparse.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 2fe4af7...efc0d25. Read the comment docs.

Copy link
Member

@jrconlin jrconlin left a comment

Choose a reason for hiding this comment

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

move these to _obsolete_args to save oremj a headache

@@ -177,17 +177,6 @@ def _add_external_router_args(parser):
"APNS settings",
type=str, default="",
env_var="APNS_CREDS")
# UDP
Copy link
Member

Choose a reason for hiding this comment

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

Move these to _obsolete_args to save oremj a headache

@bbangert bbangert dismissed jrconlin’s stale review September 7, 2017 00:42

Args have been moved.

@bbangert bbangert merged commit f1e6470 into master Sep 7, 2017
@bbangert bbangert deleted the feat/issue-886 branch September 7, 2017 17:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants