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

Error in YAML configuration docs: default value for python.setup_py_install should be true #3334

Closed
jonrkarr opened this issue Nov 30, 2017 · 5 comments

Comments

@jonrkarr
Copy link

Regarding the YAML configuration docs (http://docs.readthedocs.io/en/latest/yaml-config.html), the default for python.setup_py_install appears to be true rather than false which is stated in the documentation

@RichardLitt RichardLitt added Good First Issue Good for new contributors Needed: documentation Documentation is required labels Nov 30, 2017
@RichardLitt
Copy link
Member

Sounds like a small PR is in order. :)

@jonrkarr
Copy link
Author

jonrkarr commented Dec 2, 2017

Thanks for building the conda feature! This is very helpful for documenting packages that require more complex environments.

One more suggestion: I think it would helpful to expand the conda to note to say that requirements.txt files are not used when the conda option is used. If the conda option is used, all requirements must be invoked by setup.py or stated in environment.yml.

@humitos
Copy link
Member

humitos commented Dec 3, 2017

Thanks for your report.

Perhaps there should be a mention of projects that have never had a successful build.

I would suggest you to create a new issue for this since it's different from the one reported. Thanks!

JavaDevVictoria added a commit to JavaDevVictoria/readthedocs.org that referenced this issue Dec 4, 2017
Updated python.setup_py_install to be true, as per issue 3334: readthedocs#3334
@JavaDevVictoria
Copy link

Hi, I have submitted a pull request for this issue here: #3357

@agjohnson
Copy link
Contributor

I tested that the following were all true:

  • Default model value for installing project is False
  • Default config value for installing project is False
  • Bare project doesn't attempt to install project

So, the documentation still seems correct, this option does not seem to be enabled by default. Feel free to reopen if you have a specific case that shows otherwise, but as it stands, I don't think we need to update our docs.

@agjohnson agjohnson added Status: invalid and removed Good First Issue Good for new contributors Needed: documentation Documentation is required labels Dec 14, 2017
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

No branches or pull requests

5 participants