-
Notifications
You must be signed in to change notification settings - Fork 20
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
Officially support Python 3.12 #368
Conversation
Remove setup.py code irrelevant since 3.6/3.7 were dropped in #327
@@ -13,6 +13,16 @@ development source code and as such may not be routinely kept up to date. | |||
|
|||
# __NEXT__ | |||
|
|||
This release adds support for Python version 3.12. | |||
([#TBD](https://github.com/nextstrain/cli/pull/TBD)) |
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.
need to fill this TBD in :-)
|
||
Note that this Python version support only matters if you're installing | ||
Nextstrain CLI from PyPI or Bioconda | ||
([c.f.](https://docs.nextstrain.org/projects/cli/en/8.3.0/installation/)). It |
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.
This link shouldn't be version-specific here. It should at least be version-agnostic
([c.f.](https://docs.nextstrain.org/projects/cli/en/8.3.0/installation/)). It | |
([c.f.](https://docs.nextstrain.org/projects/cli/page/installation/)). It |
or, better yet, reflect the version it's released as
([c.f.](https://docs.nextstrain.org/projects/cli/en/8.3.0/installation/)). It | |
([c.f.](https://docs.nextstrain.org/projects/cli/en/__NEXT__/installation/)). It |
(See c05b044 for how that works.)
"types-requests; python_version != '3.6'", | ||
"types-requests <=2.28.11.12; python_version == '3.6'", | ||
"types-requests", |
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.
\o/ Thanks for spotting this!
Closing in favor of #369. |
Description of proposed changes
Related issue(s)
Checklist