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

We do require pip>=22.3 for editable installs (PEP 660). #227

Merged
merged 2 commits into from
Oct 27, 2023

Conversation

jorgensd
Copy link
Sponsor Member

This does not quite resolve issues with users trying to install with an outdated pip version, as the upgrade happens at runtime, and doesn't change the current version of pip that is executed. PEP 660: https://peps.python.org/pep-0660/

This does not quite resolve issues with users trying to install with an outdated pip version, as the upgrade happens at runtime, and doesn't change the current version of pip that is executed.
@jorgensd jorgensd requested review from dham and jhale October 23, 2023 18:10
@jhale
Copy link
Member

jhale commented Oct 27, 2023

This is the best we can do I think.

@jhale jhale enabled auto-merge October 27, 2023 09:53
@jhale jhale added this pull request to the merge queue Oct 27, 2023
Merged via the queue into main with commit 6cf3e61 Oct 27, 2023
19 checks passed
@jhale jhale deleted the dokken/add-pip-requirement branch October 27, 2023 10:04
@garth-wells
Copy link
Member

The shouldn't really be done - this version of pip isn't needed for standard installs. Unnecessary dependency pinning complicates packaging.

Users who need something niche (like editable installs) can easy take care of the pip version themselves.

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

Successfully merging this pull request may close these issues.

None yet

3 participants