Skip to content

Update yaml-config.rst to not show impossible option "python: 3.6" #4918

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

Closed
wants to merge 1 commit into from

Conversation

Naereen
Copy link

@Naereen Naereen commented Nov 20, 2018

It is really misleading to show that python: 3.6 could be used (in the first example of this page), when it still cannot be used!
I tried in this change to my readthedocs.yml file, and received instantly this error by email (see this build):

Error: Problem in your project's configuration. Invalid "python.version": expected one of (2, 2.7, 3, 3.5), got 3.6

It is really misleading to show that `python: 3.6` could be used (in the first example of this page), when it still cannot be used!
I tried (SMPyBandits/SMPyBandits@b29e893#diff-e3b87b9ac78b8654ec8ee32806cfcf02R5), and received instantly this error by email (see this build, https://readthedocs.org/projects/smpybandits/builds/8126289/):
 Error: Problem in your project's configuration. Invalid "python.version": expected one of (2, 2.7, 3, 3.5), got 3.6
Naereen added a commit to SMPyBandits/SMPyBandits that referenced this pull request Nov 20, 2018
@codecov
Copy link

codecov bot commented Nov 20, 2018

Codecov Report

Merging #4918 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4918   +/-   ##
=======================================
  Coverage   76.65%   76.65%           
=======================================
  Files         158      158           
  Lines       10059    10059           
  Branches     1269     1269           
=======================================
  Hits         7711     7711           
  Misses       2007     2007           
  Partials      341      341

@stsewd
Copy link
Member

stsewd commented Nov 20, 2018

As mention in #3070 (comment), you are missing the build.image setting.

@Naereen Naereen closed this Nov 20, 2018
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