Skip to content

Commit ce8e8de

Browse files
authored
Merge pull request microsoft#1838 from microsoft/sunilsurana/issue-1834
[rush] Add common-versions.json file in potentially changed file list.
2 parents a420d74 + 3625781 commit ce8e8de

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

apps/rush-lib/src/logic/InstallManager.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -860,7 +860,10 @@ export class InstallManager {
860860
// Additionally, if they pulled an updated npm-shrinkwrap.json file from Git,
861861
// then we can't skip this install
862862
potentiallyChangedFiles.push(this._rushConfiguration.getCommittedShrinkwrapFilename(options.variant));
863-
863+
864+
// Add common-versions.json file in potentially changed file list.
865+
potentiallyChangedFiles.push(this._rushConfiguration.getCommonVersionsFilePath(options.variant));
866+
864867
if (this._rushConfiguration.packageManager === 'pnpm') {
865868
// If the repo is using pnpmfile.js, consider that also
866869
const pnpmFileFilename: string = this._rushConfiguration.getPnpmfilePath(options.variant);
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
{
2+
"changes": [
3+
{
4+
"packageName": "@microsoft/rush",
5+
"comment": "Add common-versions.json to the set of files that, when changed, will trigger reinstallation of dependencies.",
6+
"type": "none"
7+
}
8+
],
9+
"packageName": "@microsoft/rush",
10+
"email": "susurana@microsoft.com"
11+
}

0 commit comments

Comments
 (0)