Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Lazily evaluate markers to mitigate non-PEP 440 version errors #834

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

edwardpeek-crown-public

Mitigation for #774.

Some packages define markers in an order where short-circuiting the "and" expressions can skip the problematic version comparisons.

This PR changes the marker evaluation to be lazy so that the marker expressions not affecting the outcome are not evaluated.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants