-
-
Notifications
You must be signed in to change notification settings - Fork 583
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
Add support for Python 3.12 #2175
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2175 +/- ##
=======================================
Coverage 99.15% 99.15%
=======================================
Files 38 39 +1
Lines 3089 3089
Branches 748 748
=======================================
Hits 3063 3063
Misses 15 15
Partials 11 11 |
Get this today after upgrading to py312. I can continue to use
|
.github/workflows/integration.yml
Outdated
@@ -16,18 +16,18 @@ jobs: | |||
python-version: ["3.11"] | |||
|
|||
steps: | |||
- uses: actions/checkout@v2 |
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.
Please all those action versions changes are not relevant for supporting 3.12. Just keep relevant changes in this PR.
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.
Sure, would you like them in another PR?
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.
Even better would be automate version update PRs with:
https://github.com/staticdev/ansible-role-pyenv/blob/main/.github/dependabot.yml#L4
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.
Yes, much better idea :)
.github/workflows/integration.yml
Outdated
@@ -16,18 +16,18 @@ jobs: | |||
python-version: ["3.11"] | |||
|
|||
steps: | |||
- uses: actions/checkout@v2 |
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.
Even better would be automate version update PRs with:
https://github.com/staticdev/ansible-role-pyenv/blob/main/.github/dependabot.yml#L4
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.
Thanks a lot @hugovk I just have a final comment on your PR, other than that LGTM.
Wouldn't it be better to actually add support for 3.12 instead of just pretending? (#2175 (comment)) |
Ah right, we need to update and run I did so and got this error: Traceback (most recent call last):
File "/private/tmp/isort/scripts/mkstdlibs.py", line 40, in <module>
invdata = fetch_inventory(FakeApp(), "", url)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/sphinx/ext/intersphinx.py", line 183, in fetch_inventory
f = _read_from_url(inv, config=app.config)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/sphinx/ext/intersphinx.py", line 140, in _read_from_url
_tls_info=(config.tls_verify, config.tls_cacerts))
^^^^^^^^^^^^^^^^^^
AttributeError: ('intersphinx inventory %r not fetchable due to %s: %s', 'https://docs.python.org/2.7/objects.inv', <class 'AttributeError'>, "'FakeConfig' object has no attribute 'tls_cacerts'") Downgrading from latest Sphinx 7.2.6 to 6.2.1 worked fine. It also updated the 3.11 list. |
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.
I just added a new suggestion and if possible could you rebase since we merged some other changes?
Also remove their old files from https://github.com/PyCQA/isort/tree/main/isort/stdlibs? |
py2 and py27 yes. |
I decided not to remove them after all, a lot of tests would need updating so I think it's out of scope for this PR. |
The second Python 3.12 release candidate is out! 🚀
Python 3.12.0 final is due out in two weeks: 2023-10-02.
See also https://dev.to/hugovk/help-test-python-312-beta-1508/