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

Move more content out of cli/pip_install.rst #10899

Merged
merged 6 commits into from
Feb 26, 2022

Conversation

pradyunsg
Copy link
Member

Toward #9475

See commit description for details, on commits that have them.

This should make this information more discoverable.
@pradyunsg pradyunsg changed the title Move content out of cli/pip_install.rst + switch to rubrics for reorganised content Move more content out of cli/pip_install.rst Feb 18, 2022
This should make this information more discoverable, while also
providing clearer guidance on what the difference between a "regular"
install vs an "editable" install is.
This also adds some prominent warnings to the section, to discourage
users from using these options.
This moves the content about hash checking mode into here, along with
adding minor additional recommendation of using `--no-binary :all:`.
@pradyunsg pradyunsg marked this pull request as ready for review February 18, 2022 14:09
@pradyunsg pradyunsg requested a review from a team February 18, 2022 14:54
@pradyunsg
Copy link
Member Author

Are there any outstanding concerns with this?

@pradyunsg
Copy link
Member Author

Actually, I'm gonna merge this, in line with other PRs that were a part of #9475 -- if there's concerns, holler over in #9475.

@pradyunsg pradyunsg merged commit 82aebdb into pypa:main Feb 26, 2022
@pradyunsg pradyunsg deleted the docs/more-reference branch February 26, 2022 12:39
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants