Skip to content

Commit 019b029

Browse files
committed
fix: Standardize the option name for dependency invalidation.
TL;DR Dependency invalidation wasn't getting enabled automatically when it should have been. We were reading the value out of pluginInfo as dependencyInvalidation but putting the value into pluginInfo as hasDependencyInvalidation. Since we use the option dependencyInvalidation elsewhere, I've standardized on that.
1 parent 92182c7 commit 019b029

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

lib/utils.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -267,7 +267,7 @@ function setupPlugins(wrappers) {
267267
parallelConfigs,
268268
canParallelize,
269269
unparallelizableWrappers,
270-
hasDependencyInvalidation: !!dependencyInvalidation,
270+
dependencyInvalidation: !!dependencyInvalidation,
271271
};
272272
}
273273

node-tests/utils_test.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ describe('utils', function() {
4646
parallelConfigs: [],
4747
canParallelize: true,
4848
unparallelizableWrappers: [],
49-
hasDependencyInvalidation: false,
49+
dependencyInvalidation: false,
5050
});
5151
});
5252

@@ -76,7 +76,7 @@ describe('utils', function() {
7676
parallelConfigs: ['something', 'something else'],
7777
canParallelize: true,
7878
unparallelizableWrappers: [],
79-
hasDependencyInvalidation: false,
79+
dependencyInvalidation: false,
8080
});
8181
});
8282

@@ -107,7 +107,7 @@ describe('utils', function() {
107107
parallelConfigs: ['something'],
108108
canParallelize: false,
109109
unparallelizableWrappers: ['second'],
110-
hasDependencyInvalidation: false,
110+
dependencyInvalidation: false,
111111
});
112112
});
113113
});

0 commit comments

Comments
 (0)