Skip to content

README: Correct Python version compatibility#259

Open
akx wants to merge 1 commit intorubik:masterfrom
akx:readme-fix-py-compat
Open

README: Correct Python version compatibility#259
akx wants to merge 1 commit intorubik:masterfrom
akx:readme-fix-py-compat

Conversation

@akx
Copy link
Copy Markdown

@akx akx commented Dec 15, 2023

The current versions of Radon do not run on Python 2.

docker run -it python:2.7.10 sh -c 'pip install radon && radon'
Successfully installed colorama-0.4.1 funcsigs-1.0.2 mando-0.7.1 radon-6.0.1 six-1.16.0
$
Traceback (most recent call last):
  File "/usr/local/bin/radon", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python2.7/site-packages/radon/__init__.py", line 10, in main
    from radon.cli import program, log_error
  File "/usr/local/lib/python2.7/site-packages/radon/cli/__init__.py", line 23, in <module>
    from radon.cli.harvest import (
  File "/usr/local/lib/python2.7/site-packages/radon/cli/harvest.py", line 6, in <module>
    from builtins import super
ImportError: No module named builtins
$

@kieran-ryan
Copy link
Copy Markdown

kieran-ryan commented Dec 23, 2024

@akx, I have opened a pull request dropping Python 2 support and extending to Python 3.13; though kept 3.3-3.6 as was able to run the package locally with earlier versions and ensure of permutations of removal for this package. Interested in any feedback.

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