-
Notifications
You must be signed in to change notification settings - Fork 80
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
[MRG] Remove Python 2.7 support (& end Python 2 compatibility) #1145
Conversation
Codecov Report
@@ Coverage Diff @@
## latest #1145 +/- ##
=========================================
Coverage ? 92.55%
=========================================
Files ? 73
Lines ? 5764
Branches ? 0
=========================================
Hits ? 5335
Misses ? 429
Partials ? 0 Continue to review full report at Codecov.
|
We can also check any place where |
(and remove 2.7 from CI) |
(this turns out to require some actual work :) |
README.md
Outdated
@@ -54,7 +54,7 @@ A quickstart tutorial [is available](https://sourmash.readthedocs.io/en/latest/t | |||
|
|||
### Requirements | |||
|
|||
sourmash runs under both Python 2.7.x and Python 3.5+. The base | |||
sourmash runs under Python 3.7 and later. The base |
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.
oooh, 3.7? Sounds good! =]
Remove all of the compatibility code and version checking necessary to support Python 2.7. 🎉
Fixes #1033
Fixes #318
Fixes #1043
make test
Did it pass the tests?make coverage
Is the new code covered?without a major version increment. Changing file formats also requires a
major version number increment.
changes were made?