Skip to content

Commit 004c0ee

Browse files
committed
fix(release-version): change includeAll for includeAllDependencies
1 parent d38f025 commit 004c0ee

File tree

1 file changed

+17
-4
lines changed

1 file changed

+17
-4
lines changed

packages/rust/src/generators/release-version/release-version.ts

Lines changed: 17 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -588,17 +588,30 @@ function resolveLocalPackageDependencies(
588588
filteredProjects: ProjectGraphProjectNode[],
589589
projectNameToPackageRootMap: Map<string, string>,
590590
resolvePackageRoot: (projectNode: ProjectGraphProjectNode) => string,
591-
includeAll = false
591+
includeAllDependencies = false
592592
): Record<string, LocalPackageDependency[]> {
593593
const localPackageDependencies: Record<string, LocalPackageDependency[]> = {};
594594

595-
const projects = includeAll
596-
? Object.values(projectGraph.nodes)
595+
// Recursively retrieves all implicit dependencies for a given project
596+
function retrieveDeps(projectName: string): string[] {
597+
const deps =
598+
projectGraph.nodes[projectName]?.data?.implicitDependencies || [];
599+
600+
return [...deps, ...deps.flatMap(retrieveDeps)];
601+
}
602+
603+
// if includeAllDependencies is true, we want to include all dependencies of filtered projects, otherwise we only want to include the filtered projects
604+
const projects = includeAllDependencies
605+
? filteredProjects
606+
.map((p) => p.name)
607+
.flatMap(retrieveDeps)
608+
.map((dep) => projectGraph.nodes[dep])
609+
.filter(Boolean)
597610
: filteredProjects;
598611

599612
for (const projectNode of projects) {
600613
// Ensure that the packageRoot is resolved for the project and added to the map for later use
601-
if (includeAll) {
614+
if (includeAllDependencies) {
602615
fillPackageRootMap(
603616
projectNameToPackageRootMap,
604617
projectNode,

0 commit comments

Comments
 (0)