Skip to content
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

Merged
merged 2 commits into from
Sep 21, 2022
Merged

Conversation

ccbaumler
Copy link
Contributor

@ccbaumler ccbaumler commented Sep 20, 2022

Fixes #2285

  • Removed python version from line 78 of README.md and changed python version to 3.10 in doc/release.md

@codecov
Copy link

codecov bot commented Sep 20, 2022

Codecov Report

Merging #2286 (3f0e71f) into latest (5540346) will increase coverage by 7.32%.
The diff coverage is n/a.

@@            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              
Flag Coverage Δ
python 92.14% <ø> (ø)
rust ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/core/src/ffi/hyperloglog.rs
src/core/src/index/linear.rs
src/core/src/sketch/hyperloglog/estimators.rs
src/core/src/sketch/nodegraph.rs
src/core/src/index/sbt/mod.rs
src/core/src/ffi/cmd/compute.rs
src/core/tests/minhash.rs
src/core/src/ffi/utils.rs
src/core/src/ffi/minhash.rs
src/core/src/storage.rs
... and 21 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ccbaumler ccbaumler changed the title Updated python version [MRG] Updated python version Sep 20, 2022
Copy link
Contributor

@ctb ctb left a 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!

README.md Outdated Show resolved Hide resolved
@@ -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
Copy link
Contributor

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>
@ctb ctb changed the title [MRG] Updated python version [MRG] Update python version per deprecation policy Sep 21, 2022
@ccbaumler ccbaumler merged commit c202b6b into latest Sep 21, 2022
@ccbaumler ccbaumler deleted the update-python branch September 21, 2022 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update python version in install instructions
2 participants