-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Support Python 3 #2417
Support Python 3 #2417
Conversation
074b603
to
ed6771a
Compare
Codecov Report
@@ Coverage Diff @@
## master #2417 +/- ##
==========================================
+ Coverage 83.91% 89.83% +5.91%
==========================================
Files 580 11 -569
Lines 34339 590 -33749
Branches 4175 80 -4095
==========================================
- Hits 28817 530 -28287
+ Misses 4270 41 -4229
+ Partials 1252 19 -1233 |
2f27293
to
2bd4478
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM ! 💯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One nit about documenting why we use PUT to ping the server instead of GET
@@ -15,47 +16,47 @@ def test_stats_for_version(): | |||
|
|||
# v0.90.5 | |||
metrics = stats_for_version([0, 90, 5]) | |||
assert len(metrics) == 127 | |||
assert len(metrics) == 124 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why has this changed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The version detection functions were forever modifying the constants and now we make proper copies,
* Support Python 3 * add comment * remove whitespace
What does this PR do?
Also