Remove double lookup

microsoft · Aug 12, 2022 · 68f7bfe · 68f7bfe
1 parent 2b8488e
commit 68f7bfe
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/libraries/rush-lib/src/logic/ProjectChangeAnalyzer.ts b/libraries/rush-lib/src/logic/ProjectChangeAnalyzer.ts
@@ -376,11 +376,11 @@ export class ProjectChangeAnalyzer {
       let i: number = 0;
       for (const projectDeps of projectHashDeps.values()) {
         const projectDependencyManifestPath: string = projectDependencyManifestPaths[i];
-        if (!hashes.has(projectDependencyManifestPath)) {
+        const hash: string | undefined = hashes.get(projectDependencyManifestPath);
+        if (hash === undefined) {
           throw new InternalError(`Expected to get a hash for ${projectDependencyManifestPath}`);
         }
 
-        const hash: string = hashes.get(projectDependencyManifestPath)!;
         projectDeps.set(projectDependencyManifestPath, hash);
         i++;
       }