-
-
Notifications
You must be signed in to change notification settings - Fork 506
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
Document how to replace deprecated indexserver configuration directive. #1976
Conversation
docs/config.rst
Outdated
(DEPRECATED, will be removed in a future version) Multi-line ``name = | ||
URL`` definitions of python package servers. Dependencies can | ||
specify using a specified index server through the | ||
(DEPRECATED, will be removed in a future version) Use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMHO a better way to specify the index server is to do:
setenv =
PIP_INDEX_URL = {env:PIP_INDEX_URL:https://magic.com/simple}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree.
I've changed the commit so there's only a note at the top, following is the description of indexserver configuration option and last is the new recommendation.
I've changed magic.com/simple to pypi.org/simple to avoid someone getting magic.com. (example.com could also be used).
The `install_command` directive lists it can configure index server, but the `indexserver` directive does not point to that. This PR adds this link. I have verified the documentation builds ok with `tox -e docs`.
docs/config.rst
Outdated
@@ -553,6 +555,17 @@ Complete list of settings that you can put into ``testenv*`` sections: | |||
will make tox install all dependencies from this PyPI index server | |||
(including when installing the project sdist package). | |||
|
|||
The recommended way to set custom index server URL is to use :conf:`setenv`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The recommended way to set custom index server URL is to use :conf:`setenv`: | |
The recommended way to set a custom index server URL is to use :conf:`setenv`: |
docs/config.rst
Outdated
PIP_INDEX_URL = {env:PIP_INDEX_URL:https://pypi.org/simple/} | ||
|
||
This will ensure the desired index server gets used for virtual environment | ||
creation and allow overriding index server URL with environment variable. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
creation and allow overriding index server URL with environment variable. | |
creation and allow overriding the index server URL with an environment variable. |
docs/config.rst
Outdated
to configure PIP_INDEX_URL environment variable, see below. | ||
|
||
Multi-line ``name = URL`` definitions of python package servers. | ||
Dependencies can specify using a specified index server through the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dependencies can specify using a specified index server through the | |
You can specify an alternative index server for dependencies by applying the |
…e. (tox-dev#1976) Co-authored-by: Gasper Zejn <zelo.zejn@gmail.com>
The
install_command
directive lists it can configure index server, but theindexserver
directive does not point to that. This PR adds this link.I have verified the documentation builds ok with
tox -e docs
.