-
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] Update python version per deprecation policy #2286
Conversation
Codecov Report
@@ Coverage Diff @@
## latest #2286 +/- ##
==========================================
+ Coverage 84.82% 92.14% +7.32%
==========================================
Files 131 100 -31
Lines 15676 11400 -4276
Branches 2251 2251
==========================================
- Hits 13297 10505 -2792
+ Misses 2085 601 -1484
Partials 294 294
Flags with carried forward coverage won't be shown. Click here to find out more. 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
Looks good! Please update the PR description so that the words
"Fixes #2285"
are on a line all by themselves before merging - that way that issue will be automatically closed upon merge. Thanks!
@@ -8,7 +8,7 @@ Michael Crusoe. | |||
The basic build environment needed below can be created as follows: | |||
|
|||
``` | |||
conda create -y -n sourmash-rc python=3.8 pip cxx-compiler make twine tox tox-conda setuptools setuptools_scm | |||
conda create -y -n sourmash-rc python=3.10 pip cxx-compiler make twine tox tox-conda setuptools setuptools_scm |
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.
yep - it's good to put the python version explicitly here, because we want to control the version when cutting a release.
Co-authored-by: C. Titus Brown <titus@idyll.org>
Fixes #2285
README.md
and changed python version to 3.10 indoc/release.md