Skip to content

Commit 384f445

Browse files
charliermarshzanieb
authored andcommitted
Check dependency groups for credentials (#8393)
## Summary We have to iterate over all user-defined dependencies here. We were missing the new `[dependency-groups]` section. Part of #8272.
1 parent 1bee823 commit 384f445

File tree

1 file changed

+17
-1
lines changed
  • crates/uv/src/commands/project

1 file changed

+17
-1
lines changed

crates/uv/src/commands/project/sync.rs

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ use uv_python::{PythonDownloads, PythonEnvironment, PythonPreference, PythonRequ
2323
use uv_resolver::{FlatIndex, Lock};
2424
use uv_types::{BuildIsolation, HashStrategy};
2525
use uv_warnings::warn_user;
26-
use uv_workspace::pyproject::{Source, Sources, ToolUvSources};
26+
use uv_workspace::pyproject::{DependencyGroupSpecifier, Source, Sources, ToolUvSources};
2727
use uv_workspace::{DiscoveryOptions, InstallTarget, MemberDiscovery, VirtualProject, Workspace};
2828

2929
use crate::commands::pip::loggers::{DefaultInstallLogger, DefaultResolveLogger, InstallLogger};
@@ -480,6 +480,21 @@ fn store_credentials_from_workspace(workspace: &Workspace) {
480480
.into_iter()
481481
.flat_map(|optional| optional.values())
482482
.flatten();
483+
let dependency_groups = member
484+
.pyproject_toml()
485+
.dependency_groups
486+
.as_ref()
487+
.into_iter()
488+
.flatten()
489+
.flat_map(|(_, dependencies)| {
490+
dependencies.iter().filter_map(|specifier| {
491+
if let DependencyGroupSpecifier::Requirement(requirement) = specifier {
492+
Some(requirement)
493+
} else {
494+
None
495+
}
496+
})
497+
});
483498
let dev_dependencies = member
484499
.pyproject_toml()
485500
.tool
@@ -491,6 +506,7 @@ fn store_credentials_from_workspace(workspace: &Workspace) {
491506

492507
for requirement in dependencies
493508
.chain(optional_dependencies)
509+
.chain(dependency_groups)
494510
.filter_map(|requires_dist| {
495511
LenientRequirement::<VerbatimParsedUrl>::from_str(requires_dist)
496512
.map(Requirement::from)

0 commit comments

Comments
 (0)