-
-
Notifications
You must be signed in to change notification settings - Fork 615
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
Fix py27 failing jobs #1151
Fix py27 failing jobs #1151
Conversation
3db53cc
to
58ff161
Compare
Codecov Report
@@ Coverage Diff @@
## master #1151 +/- ##
=======================================
Coverage 99.49% 99.49%
=======================================
Files 36 36
Lines 2752 2752
Branches 326 326
=======================================
Hits 2738 2738
Misses 8 8
Partials 6 6 Continue to review full report at Codecov.
|
Oof, keep |
I'm all for upgrading past these problems. Only 1 and a half questions:
|
It's not necessary for py3, but as we've seen before (#984) it's better to install a fresh |
If |
|
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.
Alright, if the py3 flow does need virtualenv installed, then yes, let's keep it fresh.
Thank you so much! 🙏 |
Refs pypa/setuptools-scm#442
virtualenv
fixed issue withpath.py
setuptools-scm-4.1.0
fixed issue with: