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

config doesn't define skipsdist #683

Closed
go8ose opened this issue Nov 28, 2017 · 2 comments
Closed

config doesn't define skipsdist #683

go8ose opened this issue Nov 28, 2017 · 2 comments
Assignees
Labels
area:documentation feature:change something exists already but should behave differently pr-merged
Milestone

Comments

@go8ose
Copy link

go8ose commented Nov 28, 2017

Hi,

I just noticed that tox/doc/config.rst mentions that skipsdist defaults to false, but it doesn't actually say what it does.

I'm sure to an experienced python developer they'd understand that it skips the step where tox runs a /setup.py sdist And the tox/doc/example/general.rst file, in the 'Avoiding expensive sdist' section spells this out.

But as someone that just googled straight into the tox/doc/config.rst page, this wasn't immediately clear to me. Some of the other global settings are defined at the bottom of this section. Perhaps all of them should be. I also noticed that skip_missing_interpreters was defined, but missing from the list of global options.

Regards,
Geoff

@cryvate
Copy link

cryvate commented Nov 28, 2017

I think in general, the section on global options could do on expansion similar to the virtualenv test environment settings so that each option has its own heading. I'll have a look later

@gaborbernat gaborbernat added area:documentation feature:change something exists already but should behave differently labels Dec 5, 2017
@cryvate
Copy link

cryvate commented Dec 13, 2017

This is still in progress: quite a few descriptions to write.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area:documentation feature:change something exists already but should behave differently pr-merged
Projects
None yet
Development

No branches or pull requests

3 participants