-
Notifications
You must be signed in to change notification settings - Fork 85
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
docs: Use 'req@url' syntax to install from remote VCS #2115
docs: Use 'req@url' syntax to install from remote VCS #2115
Conversation
Codecov ReportBase: 98.30% // Head: 98.30% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #2115 +/- ##
=======================================
Coverage 98.30% 98.30%
=======================================
Files 69 69
Lines 4531 4531
Branches 645 645
=======================================
Hits 4454 4454
Misses 45 45
Partials 32 32
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
* Use 'req@url' syntax when using pip to install from a remote git repository over 'url#egg=req' to avoid the use of #egg= fragments with a non-PEP 508 name. This will be required in pip v25.0+. - c.f. https://github.com/pypa/pip/pull/ 11617 for more details.
4e30b35
to
5e22958
Compare
Relevant RTD build: https://pyhf--2115.org.readthedocs.build/en/2115/installation.html |
I'm going to approve and merge this myself to keep things moving. As always, PRs approved by a single core dev can be reverted as needed by the rest of the dev team. |
Description
Resolves #2114
req@url
syntax when usingpip
to install from a remote Git repository overurl#egg=req
to avoid the use of#egg=
fragments with a non-PEP 508 name. This will be required inpip
v25.0+
. c.f. Restrict#egg=
fragments to valid PEP 508 names pypa/pip#11617 for more details. This avoidspip
throwing the following warning:Checklist Before Requesting Reviewer
Before Merging
For the PR Assignees: