Skip to content

Add --all-groups to uv pip install and uv pip sync #14789

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

charliermarsh
Copy link
Member

@charliermarsh charliermarsh commented Jul 21, 2025

Summary

--all-groups doesn't accept a path; instead, it assumes the pyproject.toml in the root.

Closes #14483.

@charliermarsh charliermarsh force-pushed the charlie/all-groups branch 2 times, most recently from ecd66d4 to a5b3839 Compare July 21, 2025 14:34
@charliermarsh charliermarsh temporarily deployed to uv-test-registries July 21, 2025 14:36 — with GitHub Actions Inactive
@charliermarsh charliermarsh marked this pull request as ready for review July 21, 2025 14:57
@charliermarsh charliermarsh temporarily deployed to uv-test-registries July 21, 2025 14:57 — with GitHub Actions Inactive
@charliermarsh charliermarsh requested review from Gankra and zanieb July 21, 2025 14:57
@charliermarsh charliermarsh added enhancement New feature or improvement to existing functionality cli Related to the command line interface labels Jul 21, 2025
Comment on lines +1862 to +1863
#[arg(long, overrides_with("all_groups"), hide = true)]
pub no_all_groups: bool,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting, we don't have this anywhere else right now.

Comment on lines +9760 to +9761
.arg("-r").arg("pyproject.toml")
.arg("--all-groups"), @r"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems misleading, the -r shouldn't be needed there, right?

Comment on lines +11609 to +11611
.arg("-r")
.arg("pylock.toml")
.arg("--all-groups"), @r"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gosh, I was not aware we added -r pylock.toml --group support. That's sort of concerning, since it doesn't match the existing --group semantics (i.e., not applying to other input files)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli Related to the command line interface enhancement New feature or improvement to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support --all-groups for uv pip install
2 participants