-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Change Python environment default to 3 #3070
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
Comments
The code to be changed is here: https://github.com/rtfd/readthedocs.org/blob/master/readthedocs/projects/models.py#L200 |
Hi, |
@Naereen python 3.6 is already supported, you need to use a configuration file https://docs.readthedocs.io/en/latest/yaml-config.html |
Thanks @stsewd ! |
@ericholscher I think we can do this now. I wonder if we need to create a date Feature flag for this, so only new projects start building with |
Oh, I got confused. By changing the |
Hi! I'm quite curious, I though python 3.6 was supported, but for instance this build tried to use |
It's supported, please see https://docs.readthedocs.io/en/latest/yaml-config.html#python-version
You need build:
image: latest |
Oh, OK. Sorry I (still) didn't understand this point. Thanks! |
We use docker images to encapsulate the build process, each docker image has different python versions, and libs, here is the list https://docs.readthedocs.io/en/latest/builds.html#the-build-environment. Using latest is recommendable. |
Is there any reason why 3.6 isn't currently the default? |
Just legacy code. We are upgrading it to make this to happen soon. |
Python 2 is going to stop being maintained in less than 3 years. We should switch our default Python version for projects to Python 3, as this should increase the number of python 3 projects being built on RTD by a large number. It's just a small settings change, but putting it here for discussion.
The text was updated successfully, but these errors were encountered: