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

A number of additional fixes for newer python versions #2057

Merged
merged 3 commits into from
Apr 4, 2025

Conversation

rpgoldman
Copy link
Contributor

Most of these are actually due to Christopher Genovese in discussion of #2051

The most recent pull request fixes some of the 3.12 incompatibilities but some remain.

This fixes the startup python string in elpy.el and also some fixes to the black and jedi interfaces. Finally, it updates requirements.txt with a conditional load of distutils or packaging.

PR checklist

Please make sure that the following things have been addressed (and check the relevant checkboxes):

  • Commits respect our guidelines
  • Tests are passing properly (see here on how to run Elpy's tests)

Tests are broken beyond my ability to fix them.

This python string tried to load `LooseVersion` from distutils, but
that has been removed from python as of 3.12.

This version either imports from `distutils` or from `packaging` if
that fails.

Fix from Christopher Genovese:
jorgenschaefer#2051 (comment)
Previous requirements.txt called for setuptools, which will only work
before python 3.12.
Fix conditional loading of version on python >= 3.12.
@gopar gopar merged commit 0b381f5 into jorgenschaefer:master Apr 4, 2025
0 of 35 checks passed
@gopar
Copy link
Collaborator

gopar commented Apr 4, 2025

Thank you for the PR

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.

2 participants