-
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
Upgrade to Python 3.7 for tests #3243
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3243 +/- ##
=========================================
- Coverage 85.24% 84.7% -0.55%
=========================================
Files 684 694 +10
Lines 36272 37035 +763
Branches 4153 4262 +109
=========================================
+ Hits 30920 31370 +450
- Misses 4164 4393 +229
- Partials 1188 1272 +84 |
Codecov Report
@@ Coverage Diff @@
## master #3243 +/- ##
==========================================
+ Coverage 85.24% 85.35% +0.11%
==========================================
Files 684 684
Lines 36272 36279 +7
Branches 4153 4318 +165
==========================================
+ Hits 30920 30967 +47
+ Misses 4164 4121 -43
- Partials 1188 1191 +3 |
52ca1f2
to
7fc7bf6
Compare
@@ -1,15 +1,13 @@ | |||
[tox] | |||
minversion = 2.0 | |||
basepython = py36 | |||
basepython = py27 |
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.
Should this be py37?
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.
Only Agent 5 so py2
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, thanks!
Motivation
🐍 3️⃣
Notes
flake8
on 3.7 exposed some things that were not caught on 2.x, for some reason. I fixed those tooNotes
Certificate handling in the
http_check
does not work on 3.7 so that still uses 3.6. We'll fix that in a subsequent pr