fix(workspace): topoligically sort projects for affected:build --all

Closes #1302 Aligns behaviour of `affected --base=[SHA] --head=[SHA]` and `affected --all`
nrwl · May 2, 2019 · 85351a3 · 85351a3
1 parent 0afcbb3
commit 85351a3
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/packages/workspace/src/command-line/affected.ts b/packages/workspace/src/command-line/affected.ts
@@ -13,7 +13,7 @@ import {
   getAllLibNames,
   getProjectNames,
   parseFiles,
-  getAllProjectNamesWithTarget,
+  getAllProjectsWithTarget,
   getAffectedProjectsWithTarget,
   readAngularJson
 } from './shared';
@@ -123,7 +123,7 @@ function getProjects(
   all: boolean
 ) {
   const projects = all
-    ? getAllProjectNamesWithTarget(target)
+    ? getAllProjectsWithTarget(target)
     : getAffectedProjectsWithTarget(target)(parseFiles(parsedArgs).files);
 
   return projects

diff --git a/packages/workspace/src/command-line/shared.ts b/packages/workspace/src/command-line/shared.ts
@@ -349,6 +349,16 @@ export function getAllProjectNamesWithTarget(target: string) {
   return getProjectNames(p => p.architect[target]);
 }
 
+export function getAllProjectsWithTarget(target: string) {
+  const angularJson = readAngularJson();
+  const nxJson = readNxJson();
+  const projects = getProjectNodes(angularJson, nxJson);
+  const dependencies = readDependencies(nxJson.npmScope, projects);
+  const sortedProjects = topologicallySortProjects(projects, dependencies);
+
+  return sortedProjects.filter(p => p.architect[target]).map(p => p.name);
+}
+
 export function getProjectNames(
   predicate?: (projectNode: ProjectNode) => boolean
 ): string[] {