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

Issue 5273 unable to install extras because constraints are not recognized #5274

Merged
merged 4 commits into from
Aug 18, 2022

Conversation

dqkqd
Copy link
Contributor

@dqkqd dqkqd commented Aug 18, 2022

The issue

This fixes #5273

The fix

Convert Requirement into InstallRequirement before checking constraint.

The checklist

  • Associated issue
  • A news fragment in the news/ directory to describe this fix with the extension .bugfix.rst, .feature.rst, .behavior.rst, .doc.rst. .vendor.rst. or .trivial.rst (this will appear in the release changelog). Use semantic line breaks and name the file after the issue number or the PR #.

@matteius
Copy link
Member

@dqkqd Would you add a news fragment?

@dqkqd
Copy link
Contributor Author

dqkqd commented Aug 18, 2022

@dqkqd Would you add a news fragment?

Is a new fragment is required? I thought it’s just a bug from my previous PR. If it’s necessary, I will add that after getting back home.

@matteius
Copy link
Member

@dqkqd because it made it into a pypi release, we should include a news fragment. Since you are away right now I am going to merge this one and add a news fragment on your behalf, since I just bumped up against this issue.

@matteius matteius merged commit 640612d into pypa:main Aug 18, 2022
@dqkqd
Copy link
Contributor Author

dqkqd commented Aug 18, 2022

@matteius , thank you.

@matteius matteius changed the title Issue 5273 constraints are not recognized Issue 5273 unable to install extras because constraints are not recognized Aug 19, 2022
@dqkqd dqkqd deleted the Issue-5273-constraints-are-not-recognized branch August 28, 2022 04:48
yeisonvargasf pushed a commit to yeisonvargasf/pipenv that referenced this pull request Nov 19, 2022
* Add more test for get_constraints_from_deps
* Use pip's check_invalid_constraint_type.
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.

Pipenv lock fails for packages with extras as of 2022.8.13
2 participants