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

README: Correct Python version compatibility #259

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

akx
Copy link

@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

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