Skip to content

Commit c184a03

Browse files
committed
perf: add additional debug point for commit analyzer
1 parent c5e5eda commit c184a03

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

lib/createInlinePluginCreator.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,7 @@ function createInlinePluginCreator(packages, multiContext, synchronizer, flags)
102102

103103
// Set nextType for package from plugins.
104104
pkg._nextType = await plugins.analyzeCommits(context);
105+
pkg._rawNextType = pkg._nextType;
105106

106107
// Wait until all todo packages have been analyzed.
107108
pkg._analyzed = true;
@@ -130,7 +131,8 @@ function createInlinePluginCreator(packages, multiContext, synchronizer, flags)
130131
}
131132

132133
debug("commits analyzed: %s", pkg.name);
133-
debug("release type: %s", pkg._nextType);
134+
debug("release type (semrel): %s", pkg._rawNextType);
135+
debug("release type (msr): %s", pkg._nextType);
134136

135137
// Return type.
136138
return pkg._nextType;

lib/getSynchronizer.js

+3-6
Original file line numberDiff line numberDiff line change
@@ -51,13 +51,10 @@ const getSynchronizer = (packages) => {
5151
// Status sync point.
5252
const waitForAll = (probe, filter = identity) => {
5353
const promise = once(probe);
54+
const awaitedPkgs = todo().filter(filter);
5455

55-
if (
56-
todo()
57-
.filter(filter)
58-
.every((p) => p.hasOwnProperty(probe))
59-
) {
60-
debug("ready: %s", probe);
56+
if (awaitedPkgs.every((p) => p.hasOwnProperty(probe))) {
57+
awaitedPkgs.length && debug("ready: %s", probe);
6158
emit(probe);
6259
}
6360

lib/updateDeps.js

+3-2
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ const _nextPreHighestVersion = (latestTag, lastVersion, pkgPreRelease) => {
127127
/**
128128
* Returns the 'highest' type of release update, major > minor > patch > undefined.
129129
* @param {...string} releaseTypes types (patch | minor | major | undefined) of which the highest to return.
130-
* @returns {string} release type considered highest
130+
* @returns {string | undefined} release type considered highest
131131
*/
132132
const getHighestReleaseType = (...releaseTypes) =>
133133
["major", "minor", "patch"].find((type) => releaseTypes.includes(type));
@@ -155,8 +155,9 @@ const resolveReleaseType = (pkg, bumpStrategy = "override", releaseStrategy = "p
155155
!pkg._lastRelease || //not released yet
156156
pkg._depsChanged.length === 0 || //no deps available
157157
pkg._depsChanged.every((dep) => dep._lastRelease && !dep._nextType) //no new deps or deps upgraded
158-
)
158+
) {
159159
return pkg._nextType;
160+
}
160161

161162
//find highest release type if strategy is inherit, starting of type set by commit analyzer
162163
if (releaseStrategy === "inherit") {

0 commit comments

Comments
 (0)