Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upGitHub is where the world builds software
Millions of developers and companies build, ship, and maintain their software on GitHub — the largest and most advanced development platform in the world.
Py38 tox env #1752
Py38 tox env #1752
Conversation
Set version
Add Python3.8 to Appveyor config
Codecov Report
@@ Coverage Diff @@
## master #1752 +/- ##
======================================
Coverage 92.3% 92.3%
======================================
Files 22 22
Lines 2274 2274
Branches 419 419
======================================
Hits 2099 2099
Misses 135 135
Partials 40 40
Continue to review full report at Codecov.
|
| @@ -1 +1 @@ | |||
| __version__ = "19.12.0" | |||
| __version__ = "19.12.2" | |||
seemethere
Jan 3, 2020
Member
Does the version need to be bumped for this?
Does the version need to be bumped for this?
ahopkins
Jan 3, 2020
Author
Member
Nope. It was a mistake and was meant to be bumped on a previous commit.
Nope. It was a mistake and was meant to be bumped on a previous commit.
|
We can probably merge this as soon as we get rid of the version bump. |
| @@ -21,23 +21,46 @@ matrix: | |||
| dist: xenial | |||
| sudo: true | |||
| name: "Python 3.7 without Extensions" | |||
| - env: TOX_ENV=py38 | |||
seemethere
Jan 3, 2020
Member
There has to be a better way of doing this instead of just copy / pase the config block.
Not really a blocker on this PR but just a note for us to look for in the future.
There has to be a better way of doing this instead of just copy / pase the config block.
Not really a blocker on this PR but just a note for us to look for in the future.
|
@huge-success/sanic-core-devs How do you guys want to handle this? This PR really should get merged so we have the Python 3.8 builds. BUT .... it is failing because apparently there is some work to do on Windows on 3.8. Should we break it off? Mark that one as an accepted failure? Remove Python 3.8 from appveyor tests? |
|
@ahopkins Does appveyor support marking some as expected test failures like it was done for travis where the failures are ignored for now? |
Yes. |
|
Coming back around to this - appveyor is purely windows and builds pass even if appveyor fails. I'm going to merge this in so if we have someone who wants to focus on improving windows support, which is currently best effort, and now noted as such in the documentation per #1778 |
Adds Python 3.8 to PyPI classifiers,
toxenvironments, and Travis builds