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

[python] Fix bug parsing extras #322

Merged
merged 7 commits into from
Dec 20, 2024
Merged

Conversation

blast-hardcheese
Copy link
Collaborator

Why

image

What changed

  • Break ((?:extras)?(?:version)?) into (extras)?(version?)
  • Thread PkgCoordinates through to Add instead of the limited resolution PkgSpec
  • Explicitly integrate Extra into pep440Join instead of just hoping everything works

Test plan

  • CI

@blast-hardcheese blast-hardcheese requested a review from a team as a code owner December 20, 2024 01:07
@blast-hardcheese blast-hardcheese requested review from airportyh and removed request for a team December 20, 2024 01:07
@blast-hardcheese blast-hardcheese force-pushed the dstewart/python-extras-bug branch from 2e4d327 to 28c7acf Compare December 20, 2024 01:10
@blast-hardcheese blast-hardcheese enabled auto-merge (squash) December 20, 2024 01:12
@blast-hardcheese blast-hardcheese added the bug Something isn't working label Dec 20, 2024
@blast-hardcheese blast-hardcheese merged commit 09a4de8 into main Dec 20, 2024
5 checks passed
@blast-hardcheese blast-hardcheese deleted the dstewart/python-extras-bug branch December 20, 2024 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants