-
Notifications
You must be signed in to change notification settings - Fork 645
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
Support recursive extras #1435
Support recursive extras #1435
Conversation
extra: Option<&ExtraName>, | ||
source: Option<&PackageName>, | ||
source_name: Option<&PackageName>, | ||
source_extra: Option<&ExtraName>, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just renamed and re-ordered these (especially since extra
got shadowed below).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, that's the code I was suspicious of
Are these warnings only visible with |
Yeah |
@@ -735,6 +735,76 @@ fn multiple_extras_required() { | |||
assert_installed(&context.venv, "c_502cbb59", "1.0.0", &context.temp_dir); | |||
} | |||
|
|||
/// all-extras-required | |||
/// | |||
/// Multiple optional dependencies are requested for the via an 'all' extra. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/// Multiple optional dependencies are requested for the via an 'all' extra. | |
/// Multiple optional dependencies are requested for the package via an 'all' extra. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! This file is actually generated. If you want to fix it, it's sourced from https://github.com/zanieb/packse/blob/main/scenarios/extras.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From astral-sh/uv#1435 --------- Co-authored-by: Zanie <contact@zanie.dev>
Summary
We had a guard in the resolve to avoid "self-dependencies" (as in
gps3==0.33.3
), but this guard was unintentionally filtering out recursive extras.Closes #1342.
Test Plan
Taken from #1352.