Skip to content

Commit d60ebc9

Browse files
authored
chore(nm): use for loop in buildTree fn (#4200)
* chore(nm): use for loop in buildTree fn * chore: yarn version check --interactive
1 parent 3edfba5 commit d60ebc9

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

.yarn/versions/89ab2ac5.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
releases:
2+
"@yarnpkg/nm": patch
3+
4+
declined:
5+
- "@yarnpkg/plugin-nm"
6+
- "@yarnpkg/pnpify"

packages/yarnpkg-nm/sources/buildNodeModulesTree.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -565,8 +565,7 @@ const populateNodeModulesTree = (pnp: PnpApi, hoistedTree: HoisterResult, option
565565
const segments = nodeModulesLocation.split(`/`);
566566
const nodeModulesIdx = segments.indexOf(NODE_MODULES);
567567

568-
let segCount = segments.length - 1;
569-
while (nodeModulesIdx >= 0 && segCount > nodeModulesIdx) {
568+
for (let segCount = segments.length - 1; nodeModulesIdx >= 0 && segCount > nodeModulesIdx; segCount--) {
570569
const dirPath = npath.toPortablePath(segments.slice(0, segCount).join(ppath.sep));
571570
const targetDir = toFilename(segments[segCount]);
572571

@@ -580,8 +579,6 @@ const populateNodeModulesTree = (pnp: PnpApi, hoistedTree: HoisterResult, option
580579
subdirs.dirList.add(targetDir);
581580
}
582581
}
583-
584-
segCount--;
585582
}
586583
}
587584

0 commit comments

Comments
 (0)