release-please 14.1.2 → 14.2.0

package/build/src/plugins/maven-workspace.js

@@ -19,13 +19,13 @@ const version_1 = require("../version");
 const versioning_strategy_1 = require("../versioning-strategy");
 const dom = require("@xmldom/xmldom");
 const xpath = require("xpath");
-const logger_1 = require("../util/logger");
 const path_1 = require("path");
 const pom_xml_1 = require("../updaters/java/pom-xml");
 const changelog_1 = require("../updaters/changelog");
 const pull_request_title_1 = require("../util/pull-request-title");
 const pull_request_body_1 = require("../util/pull-request-body");
 const branch_name_1 = require("../util/branch-name");
+const logger_1 = require("../util/logger");
 const JAVA_RELEASE_TYPES = new Set(['java', 'java-bom', 'java-yoshi', 'maven']);
 const XPATH_PROJECT_GROUP = '/*[local-name()="project"]/*[local-name()="groupId"]';
 const XPATH_PROJECT_ARTIFACT = '/*[local-name()="project"]/*[local-name()="artifactId"]';
@@ -37,17 +37,17 @@ class MavenWorkspace extends workspace_1.WorkspacePlugin {
         const document = new dom.DOMParser().parseFromString(content.parsedContent);
         const groupNodes = xpath.select(XPATH_PROJECT_GROUP, document);
         if (groupNodes.length === 0) {
-            logger_1.logger.warn(`Missing project.groupId in ${path}`);
+            this.logger.warn(`Missing project.groupId in ${path}`);
             return;
         }
         const artifactNodes = xpath.select(XPATH_PROJECT_ARTIFACT, document);
         if (artifactNodes.length === 0) {
-            logger_1.logger.warn(`Missing project.artifactId in ${path}`);
+            this.logger.warn(`Missing project.artifactId in ${path}`);
             return;
         }
         const versionNodes = xpath.select(XPATH_PROJECT_VERSION, document);
         if (versionNodes.length === 0) {
-            logger_1.logger.warn(`Missing project.version in ${path}`);
+            this.logger.warn(`Missing project.version in ${path}`);
             return;
         }
         const dependencies = [];
@@ -94,7 +94,7 @@ class MavenWorkspace extends workspace_1.WorkspacePlugin {
             const path = (0, path_1.dirname)(pomFile);
             const config = this.repositoryConfig[path];
             if (!config) {
-                logger_1.logger.info(`path '${path}' not configured, ignoring '${pomFile}'`);
+                this.logger.info(`path '${path}' not configured, ignoring '${pomFile}'`);
                 continue;
             }
             const mavenArtifact = await this.fetchPom(pomFile);
@@ -108,7 +108,7 @@ class MavenWorkspace extends workspace_1.WorkspacePlugin {
                     candidate;
             }
             else {
-                logger_1.logger.warn(`found ${pomFile} in path ${path}, but did not find an associated candidate PR`);
+                this.logger.warn(`found ${pomFile} in path ${path}, but did not find an associated candidate PR`);
             }
         }
         return {
@@ -145,7 +145,7 @@ class MavenWorkspace extends workspace_1.WorkspacePlugin {
             throw new Error(`Didn't find updated version for ${artifact.name}`);
         }
         const updater = new pom_xml_1.PomXml(version, updatedVersions);
-        const dependencyNotes = getChangelogDepsNotes(artifact, updater, updatedVersions);
+        const dependencyNotes = getChangelogDepsNotes(artifact, updater, updatedVersions, this.logger);
         existingCandidate.pullRequest.updates =
             existingCandidate.pullRequest.updates.map(update => {
                 if (update.path === (0, workspace_1.addPath)(existingCandidate.path, 'pom.xml')) {
@@ -154,7 +154,7 @@ class MavenWorkspace extends workspace_1.WorkspacePlugin {
                 else if (update.updater instanceof changelog_1.Changelog) {
                     if (dependencyNotes) {
                         update.updater.changelogEntry =
-                            (0, workspace_1.appendDependenciesSectionToChangelog)(update.updater.changelogEntry, dependencyNotes);
+                            (0, workspace_1.appendDependenciesSectionToChangelog)(update.updater.changelogEntry, dependencyNotes, this.logger);
                     }
                 }
                 return update;
@@ -163,13 +163,13 @@ class MavenWorkspace extends workspace_1.WorkspacePlugin {
         if (dependencyNotes) {
             if (existingCandidate.pullRequest.body.releaseData.length > 0) {
                 existingCandidate.pullRequest.body.releaseData[0].notes =
-                    (0, workspace_1.appendDependenciesSectionToChangelog)(existingCandidate.pullRequest.body.releaseData[0].notes, dependencyNotes);
+                    (0, workspace_1.appendDependenciesSectionToChangelog)(existingCandidate.pullRequest.body.releaseData[0].notes, dependencyNotes, this.logger);
             }
             else {
                 existingCandidate.pullRequest.body.releaseData.push({
                     component: artifact.name,
                     version: existingCandidate.pullRequest.version,
-                    notes: (0, workspace_1.appendDependenciesSectionToChangelog)('', dependencyNotes),
+                    notes: (0, workspace_1.appendDependenciesSectionToChangelog)('', dependencyNotes, this.logger),
                 });
             }
         }
@@ -181,14 +181,14 @@ class MavenWorkspace extends workspace_1.WorkspacePlugin {
             throw new Error(`Didn't find updated version for ${artifact.name}`);
         }
         const updater = new pom_xml_1.PomXml(version, updatedVersions);
-        const dependencyNotes = getChangelogDepsNotes(artifact, updater, updatedVersions);
+        const dependencyNotes = getChangelogDepsNotes(artifact, updater, updatedVersions, this.logger);
         const pullRequest = {
             title: pull_request_title_1.PullRequestTitle.ofTargetBranch(this.targetBranch),
             body: new pull_request_body_1.PullRequestBody([
                 {
                     component: artifact.name,
                     version,
-                    notes: (0, workspace_1.appendDependenciesSectionToChangelog)('', dependencyNotes),
+                    notes: (0, workspace_1.appendDependenciesSectionToChangelog)('', dependencyNotes, this.logger),
                 },
             ]),
             updates: [
@@ -237,13 +237,13 @@ exports.MavenWorkspace = MavenWorkspace;
 function packageNameFromGav(gav) {
     return `${gav.groupId}:${gav.artifactId}`;
 }
-function getChangelogDepsNotes(artifact, updater, updatedVersions) {
+function getChangelogDepsNotes(artifact, updater, updatedVersions, logger = logger_1.logger) {
     const document = new dom.DOMParser().parseFromString(artifact.pomContent);
     const dependencyUpdates = updater.dependencyUpdates(document, updatedVersions);
     const depUpdateNotes = [];
     for (const dependencyUpdate of dependencyUpdates) {
         depUpdateNotes.push(`\n    * ${dependencyUpdate.name} bumped to ${dependencyUpdate.version}`);
-        logger_1.logger.info(`bumped ${dependencyUpdate.name} to ${dependencyUpdate.version}`);
+        logger.info(`bumped ${dependencyUpdate.name} to ${dependencyUpdate.version}`);
     }
     if (depUpdateNotes.length > 0) {
         return `* The following workspace dependencies were updated${depUpdateNotes.join()}`;

package/build/src/plugins/merge.js

@@ -20,7 +20,6 @@ const pull_request_title_1 = require("../util/pull-request-title");
 const pull_request_body_1 = require("../util/pull-request-body");
 const branch_name_1 = require("../util/branch-name");
 const composite_1 = require("../updaters/composite");
-const logger_1 = require("../util/logger");
 /**
  * This plugin merges multiple pull requests into a single
  * release pull request.
@@ -38,7 +37,7 @@ class Merge extends plugin_1.ManifestPlugin {
         if (candidates.length < 1) {
             return candidates;
         }
-        logger_1.logger.info(`Merging ${candidates.length} pull requests`);
+        this.logger.info(`Merging ${candidates.length} pull requests`);
         const [inScopeCandidates, outOfScopeCandidates] = candidates.reduce((collection, candidate) => {
             if (candidate.config.separatePullRequests) {
                 collection[1].push(candidate);

package/build/src/plugins/node-workspace.js

@@ -16,7 +16,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.NodeWorkspace = void 0;
 const package_graph_1 = require("@lerna/package-graph");
 const package_1 = require("@lerna/package");
-const logger_1 = require("../util/logger");
 const manifest_1 = require("../manifest");
 const version_1 = require("../version");
 const raw_content_1 = require("../updaters/raw-content");
@@ -62,7 +61,7 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
             }
             const candidate = candidatesByPath.get(path);
             if (candidate) {
-                logger_1.logger.debug(`Found candidate pull request for path: ${candidate.path}`);
+                this.logger.debug(`Found candidate pull request for path: ${candidate.path}`);
                 const packagePath = (0, workspace_1.addPath)(candidate.path, 'package.json');
                 const packageUpdate = candidate.pullRequest.updates.find(update => update.path === packagePath);
                 if (packageUpdate === null || packageUpdate === void 0 ? void 0 : packageUpdate.cachedFileContents) {
@@ -79,7 +78,7 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
             }
             else {
                 const packagePath = (0, workspace_1.addPath)(path, 'package.json');
-                logger_1.logger.debug(`No candidate pull request for path: ${path} - inspect package from ${packagePath}`);
+                this.logger.debug(`No candidate pull request for path: ${path} - inspect package from ${packagePath}`);
                 const contents = await this.github.getFileContentsOnBranch(packagePath, this.targetBranch);
                 packagesByPath.set(path, new Package(contents.parsedContent, path));
             }
@@ -105,7 +104,7 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
         // Update version of the package
         const newVersion = updatedVersions.get(updatedPackage.name);
         if (newVersion) {
-            logger_1.logger.info(`Updating ${updatedPackage.name} to ${newVersion}`);
+            this.logger.info(`Updating ${updatedPackage.name} to ${newVersion}`);
             updatedPackage.version = newVersion.toString();
         }
         // Update dependency versions
@@ -113,7 +112,7 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
             const depVersion = updatedVersions.get(depName);
             if (depVersion && resolved.type !== 'directory') {
                 updatedPackage.updateLocalDependency(resolved, depVersion.toString(), '^');
-                logger_1.logger.info(`${pkg.name}.${depName} updated to ^${depVersion.toString()}`);
+                this.logger.info(`${pkg.name}.${depName} updated to ^${depVersion.toString()}`);
             }
         }
         const dependencyNotes = getChangelogDepsNotes(pkg, updatedPackage);
@@ -125,7 +124,7 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
                 else if (update.updater instanceof changelog_1.Changelog) {
                     if (dependencyNotes) {
                         update.updater.changelogEntry =
-                            (0, workspace_1.appendDependenciesSectionToChangelog)(update.updater.changelogEntry, dependencyNotes);
+                            (0, workspace_1.appendDependenciesSectionToChangelog)(update.updater.changelogEntry, dependencyNotes, this.logger);
                     }
                 }
                 return update;
@@ -134,13 +133,13 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
         if (dependencyNotes) {
             if (existingCandidate.pullRequest.body.releaseData.length > 0) {
                 existingCandidate.pullRequest.body.releaseData[0].notes =
-                    (0, workspace_1.appendDependenciesSectionToChangelog)(existingCandidate.pullRequest.body.releaseData[0].notes, dependencyNotes);
+                    (0, workspace_1.appendDependenciesSectionToChangelog)(existingCandidate.pullRequest.body.releaseData[0].notes, dependencyNotes, this.logger);
             }
             else {
                 existingCandidate.pullRequest.body.releaseData.push({
                     component: updatedPackage.name,
                     version: existingCandidate.pullRequest.version,
-                    notes: (0, workspace_1.appendDependenciesSectionToChangelog)('', dependencyNotes),
+                    notes: (0, workspace_1.appendDependenciesSectionToChangelog)('', dependencyNotes, this.logger),
                 });
             }
         }
@@ -156,14 +155,14 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
         // Update version of the package
         const newVersion = updatedVersions.get(updatedPackage.name);
         if (newVersion) {
-            logger_1.logger.info(`Updating ${updatedPackage.name} to ${newVersion}`);
+            this.logger.info(`Updating ${updatedPackage.name} to ${newVersion}`);
             updatedPackage.version = newVersion.toString();
         }
         for (const [depName, resolved] of graphPackage.localDependencies) {
             const depVersion = updatedVersions.get(depName);
             if (depVersion && resolved.type !== 'directory') {
                 updatedPackage.updateLocalDependency(resolved, depVersion.toString(), '^');
-                logger_1.logger.info(`${pkg.name}.${depName} updated to ^${depVersion.toString()}`);
+                this.logger.info(`${pkg.name}.${depName} updated to ^${depVersion.toString()}`);
             }
         }
         const dependencyNotes = getChangelogDepsNotes(pkg, updatedPackage);
@@ -175,7 +174,7 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
                 {
                     component: updatedPackage.name,
                     version,
-                    notes: (0, workspace_1.appendDependenciesSectionToChangelog)('', dependencyNotes),
+                    notes: (0, workspace_1.appendDependenciesSectionToChangelog)('', dependencyNotes, this.logger),
                 },
             ]),
             updates: [
@@ -189,7 +188,7 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
                     createIfMissing: false,
                     updater: new changelog_1.Changelog({
                         version,
-                        changelogEntry: (0, workspace_1.appendDependenciesSectionToChangelog)('', dependencyNotes),
+                        changelogEntry: (0, workspace_1.appendDependenciesSectionToChangelog)('', dependencyNotes, this.logger),
                     }),
                 },
             ],

package/build/src/plugins/workspace.d.ts

@@ -1,5 +1,6 @@
 import { ManifestPlugin } from '../plugin';
 import { CandidateReleasePullRequest, RepositoryConfig } from '../manifest';
+import { Logger } from '../util/logger';
 import { VersionsMap, Version } from '../version';
 import { GitHub } from '../github';
 export declare type DependencyGraph<T> = Map<string, DependencyNode<T>>;
@@ -11,6 +12,7 @@ export interface WorkspacePluginOptions {
     manifestPath?: string;
     updateAllPackages?: boolean;
     merge?: boolean;
+    logger?: Logger;
 }
 export interface AllPackages<T> {
     allPackages: T[];
@@ -122,5 +124,5 @@ export declare abstract class WorkspacePlugin<T> extends ManifestPlugin {
     protected buildGraphOrder(graph: DependencyGraph<T>, packageNamesToUpdate: string[]): T[];
     private visitPostOrder;
 }
-export declare function appendDependenciesSectionToChangelog(changelog: string, notes: string): string;
+export declare function appendDependenciesSectionToChangelog(changelog: string, notes: string, logger?: Logger): string;
 export declare function addPath(path: string, file: string): string;

package/build/src/plugins/workspace.js

@@ -33,16 +33,16 @@ const release_please_manifest_1 = require("../updaters/release-please-manifest")
 class WorkspacePlugin extends plugin_1.ManifestPlugin {
     constructor(github, targetBranch, repositoryConfig, options = {}) {
         var _a, _b, _c;
-        super(github, targetBranch, repositoryConfig);
+        super(github, targetBranch, repositoryConfig, options.logger);
         this.manifestPath = (_a = options.manifestPath) !== null && _a !== void 0 ? _a : manifest_1.DEFAULT_RELEASE_PLEASE_MANIFEST;
         this.updateAllPackages = (_b = options.updateAllPackages) !== null && _b !== void 0 ? _b : false;
         this.merge = (_c = options.merge) !== null && _c !== void 0 ? _c : true;
     }
     async run(candidates) {
-        logger_1.logger.info('Running workspace plugin');
+        this.logger.info('Running workspace plugin');
         const [inScopeCandidates, outOfScopeCandidates] = candidates.reduce((collection, candidate) => {
             if (!candidate.pullRequest.version) {
-                logger_1.logger.warn('pull request missing version', candidate);
+                this.logger.warn('pull request missing version', candidate);
                 return collection;
             }
             if (this.inScope(candidate)) {
@@ -53,33 +53,33 @@ class WorkspacePlugin extends plugin_1.ManifestPlugin {
             }
             return collection;
         }, [[], []]);
-        logger_1.logger.info(`Found ${inScopeCandidates.length} in-scope releases`);
+        this.logger.info(`Found ${inScopeCandidates.length} in-scope releases`);
         if (inScopeCandidates.length === 0) {
             return outOfScopeCandidates;
         }
-        logger_1.logger.info('Building list of all packages');
+        this.logger.info('Building list of all packages');
         const { allPackages, candidatesByPackage } = await this.buildAllPackages(inScopeCandidates);
-        logger_1.logger.info(`Building dependency graph for ${allPackages.length} packages`);
+        this.logger.info(`Building dependency graph for ${allPackages.length} packages`);
         const graph = await this.buildGraph(allPackages);
         const packageNamesToUpdate = this.updateAllPackages
             ? allPackages.map(this.packageNameFromPackage)
             : Object.keys(candidatesByPackage);
         const orderedPackages = this.buildGraphOrder(graph, packageNamesToUpdate);
-        logger_1.logger.info(`Updating ${orderedPackages.length} packages`);
+        this.logger.info(`Updating ${orderedPackages.length} packages`);
         const updatedVersions = new Map();
         const updatedPathVersions = new Map();
         for (const pkg of orderedPackages) {
             const packageName = this.packageNameFromPackage(pkg);
-            logger_1.logger.debug(`package: ${packageName}`);
+            this.logger.debug(`package: ${packageName}`);
             const existingCandidate = candidatesByPackage[packageName];
             if (existingCandidate) {
                 const version = existingCandidate.pullRequest.version;
-                logger_1.logger.debug(`version: ${version} from release-please`);
+                this.logger.debug(`version: ${version} from release-please`);
                 updatedVersions.set(packageName, version);
             }
             else {
                 const version = this.bumpVersion(pkg);
-                logger_1.logger.debug(`version: ${version} forced bump`);
+                this.logger.debug(`version: ${version} forced bump`);
                 updatedVersions.set(packageName, version);
                 updatedPathVersions.set(this.pathFromPackage(pkg), version);
             }
@@ -90,19 +90,19 @@ class WorkspacePlugin extends plugin_1.ManifestPlugin {
             const existingCandidate = candidatesByPackage[packageName];
             if (existingCandidate) {
                 // if already has an pull request, update the changelog and update
-                logger_1.logger.info(`Updating exising candidate pull request for ${this.packageNameFromPackage(pkg)}`);
+                this.logger.info(`Updating exising candidate pull request for ${this.packageNameFromPackage(pkg)}`);
                 const newCandidate = this.updateCandidate(existingCandidate, pkg, updatedVersions);
                 newCandidates.push(newCandidate);
             }
             else {
                 // otherwise, build a new pull request with changelog and entry update
-                logger_1.logger.info(`Creating new candidate pull request for ${this.packageNameFromPackage(pkg)}`);
+                this.logger.info(`Creating new candidate pull request for ${this.packageNameFromPackage(pkg)}`);
                 const newCandidate = this.newCandidate(pkg, updatedVersions);
                 newCandidates.push(newCandidate);
             }
         }
         if (this.merge) {
-            logger_1.logger.info(`Merging ${newCandidates.length} in-scope candidates`);
+            this.logger.info(`Merging ${newCandidates.length} in-scope candidates`);
             const mergePlugin = new merge_1.Merge(this.github, this.targetBranch, this.repositoryConfig);
             newCandidates = await mergePlugin.run(newCandidates);
         }
@@ -115,7 +115,7 @@ class WorkspacePlugin extends plugin_1.ManifestPlugin {
                 versionsMap: updatedPathVersions,
             }),
         });
-        logger_1.logger.info(`Post-processing ${newCandidates.length} in-scope candidates`);
+        this.logger.info(`Post-processing ${newCandidates.length} in-scope candidates`);
         newCandidates = this.postProcessCandidates(newCandidates, updatedVersions);
         return [...outOfScopeCandidates, ...newCandidates];
     }
@@ -149,7 +149,7 @@ class WorkspacePlugin extends plugin_1.ManifestPlugin {
      *   being updated.
      */
     buildGraphOrder(graph, packageNamesToUpdate) {
-        logger_1.logger.info(`building graph order, existing package names: ${packageNamesToUpdate}`);
+        this.logger.info(`building graph order, existing package names: ${packageNamesToUpdate}`);
         // invert the graph so it's dependency name => packages that depend on it
         const dependentGraph = this.invertGraph(graph);
         const visited = new Set();
@@ -168,31 +168,31 @@ class WorkspacePlugin extends plugin_1.ManifestPlugin {
         }
         const node = graph.get(name);
         if (!node) {
-            logger_1.logger.warn(`Didn't find node: ${name} in graph`);
+            this.logger.warn(`Didn't find node: ${name} in graph`);
             return;
         }
         const nextPath = [...path, name];
         for (const depName of node.deps) {
             const dep = graph.get(depName);
             if (!dep) {
-                logger_1.logger.warn(`dependency not found in graph: ${depName}`);
+                this.logger.warn(`dependency not found in graph: ${depName}`);
                 return;
             }
             this.visitPostOrder(graph, depName, visited, nextPath);
         }
         if (!visited.has(node.value)) {
-            logger_1.logger.debug(`marking ${name} as visited and adding ${this.packageNameFromPackage(node.value)} to order`);
+            this.logger.debug(`marking ${name} as visited and adding ${this.packageNameFromPackage(node.value)} to order`);
             visited.add(node.value);
         }
     }
 }
 exports.WorkspacePlugin = WorkspacePlugin;
 const DEPENDENCY_HEADER = new RegExp('### Dependencies');
-function appendDependenciesSectionToChangelog(changelog, notes) {
+function appendDependenciesSectionToChangelog(changelog, notes, logger = logger_1.logger) {
     if (!changelog) {
         return `### Dependencies\n\n${notes}`;
     }
-    logger_1.logger.info('appending dependency notes to changelog');
+    logger.info('appending dependency notes to changelog');
     const newLines = [];
     let seenDependenciesSection = false;
     let seenDependencySectionSpacer = false;

package/build/src/strategies/base.d.ts

@@ -10,6 +10,7 @@ import { Version, VersionsMap } from '../version';
 import { TagName } from '../util/tag-name';
 import { Release } from '../release';
 import { ReleasePullRequest } from '../release-pull-request';
+import { Logger } from '../util/logger';
 import { PullRequestBody } from '../util/pull-request-body';
 import { PullRequest } from '../pull-request';
 export interface BuildUpdatesOptions {
@@ -45,6 +46,7 @@ export interface BaseStrategyOptions {
     versionFile?: string;
     snapshotLabels?: string[];
     skipSnapshot?: boolean;
+    logger?: Logger;
 }
 /**
  * A strategy is responsible for determining which files are
@@ -53,6 +55,7 @@ export interface BaseStrategyOptions {
 export declare abstract class BaseStrategy implements Strategy {
     readonly path: string;
     protected github: GitHub;
+    protected logger: Logger;
     protected component?: string;
     private packageName?;
     readonly versioningStrategy: VersioningStrategy;

package/build/src/strategies/base.js

@@ -37,14 +37,16 @@ const DEFAULT_CHANGELOG_PATH = 'CHANGELOG.md';
  */
 class BaseStrategy {
     constructor(options) {
-        var _a, _b;
+        var _a, _b, _c;
+        this.logger = (_a = options.logger) !== null && _a !== void 0 ? _a : logger_1.logger;
         this.path = options.path || manifest_1.ROOT_PROJECT_PATH;
         this.github = options.github;
         this.packageName = options.packageName;
         this.component =
             options.component || this.normalizeComponent(this.packageName);
         this.versioningStrategy =
-            options.versioningStrategy || new default_1.DefaultVersioningStrategy({});
+            options.versioningStrategy ||
+                new default_1.DefaultVersioningStrategy({ logger: this.logger });
         this.targetBranch = options.targetBranch;
         this.repository = options.github.repository;
         this.changelogPath = options.changelogPath || DEFAULT_CHANGELOG_PATH;
@@ -55,8 +57,8 @@ class BaseStrategy {
         this.releaseAs = options.releaseAs;
         this.changelogNotes =
             options.changelogNotes || new default_2.DefaultChangelogNotes(options);
-        this.includeComponentInTag = (_a = options.includeComponentInTag) !== null && _a !== void 0 ? _a : true;
-        this.includeVInTag = (_b = options.includeVInTag) !== null && _b !== void 0 ? _b : true;
+        this.includeComponentInTag = (_b = options.includeComponentInTag) !== null && _b !== void 0 ? _b : true;
+        this.includeVInTag = (_c = options.includeVInTag) !== null && _c !== void 0 ? _c : true;
         this.pullRequestTitlePattern = options.pullRequestTitlePattern;
         this.pullRequestHeader = options.pullRequestHeader;
         this.extraFiles = options.extraFiles || [];
@@ -135,18 +137,18 @@ class BaseStrategy {
      *   open a pull request.
      */
     async buildReleasePullRequest(commits, latestRelease, draft, labels = []) {
-        const conventionalCommits = await this.postProcessCommits((0, commit_1.parseConventionalCommits)(commits));
-        logger_1.logger.info(`Considering: ${conventionalCommits.length} commits`);
+        const conventionalCommits = await this.postProcessCommits((0, commit_1.parseConventionalCommits)(commits, this.logger));
+        this.logger.info(`Considering: ${conventionalCommits.length} commits`);
         if (conventionalCommits.length === 0) {
-            logger_1.logger.info(`No commits for path: ${this.path}, skipping`);
+            this.logger.info(`No commits for path: ${this.path}, skipping`);
             return undefined;
         }
         const newVersion = await this.buildNewVersion(conventionalCommits, latestRelease);
         const versionsMap = await this.updateVersionsMap(await this.buildVersionsMap(conventionalCommits), conventionalCommits, newVersion);
         const component = await this.getComponent();
-        logger_1.logger.debug('component:', component);
+        this.logger.debug('component:', component);
         const newVersionTag = new tag_name_1.TagName(newVersion, this.includeComponentInTag ? component : undefined, this.tagSeparator, this.includeVInTag);
-        logger_1.logger.debug('pull request title pattern:', this.pullRequestTitlePattern);
+        this.logger.debug('pull request title pattern:', this.pullRequestTitlePattern);
         const pullRequestTitle = pull_request_title_1.PullRequestTitle.ofComponentTargetBranchVersion(component || '', this.targetBranch, newVersion, this.pullRequestTitlePattern);
         const branchComponent = await this.getBranchComponent();
         const branchName = branchComponent
@@ -154,7 +156,7 @@ class BaseStrategy {
             : branch_name_1.BranchName.ofTargetBranch(this.targetBranch);
         const releaseNotesBody = await this.buildReleaseNotes(conventionalCommits, newVersion, newVersionTag, latestRelease, commits);
         if (this.changelogEmpty(releaseNotesBody)) {
-            logger_1.logger.info(`No user facing commits found since ${latestRelease ? latestRelease.sha : 'beginning of time'} - skipping`);
+            this.logger.info(`No user facing commits found since ${latestRelease ? latestRelease.sha : 'beginning of time'} - skipping`);
             return undefined;
         }
         const updates = await this.buildUpdates({
@@ -225,7 +227,7 @@ class BaseStrategy {
     }
     async buildNewVersion(conventionalCommits, latestRelease) {
         if (this.releaseAs) {
-            logger_1.logger.warn(`Setting version for ${this.path} from release-as configuration`);
+            this.logger.warn(`Setting version for ${this.path} from release-as configuration`);
             return version_1.Version.parse(this.releaseAs);
         }
         const releaseAsCommit = conventionalCommits.find(conventionalCommit => conventionalCommit.notes.find(note => note.title === 'RELEASE AS'));
@@ -244,7 +246,7 @@ class BaseStrategy {
         return new Map();
     }
     async parsePullRequestBody(pullRequestBody) {
-        return pull_request_body_1.PullRequestBody.parse(pullRequestBody);
+        return pull_request_body_1.PullRequestBody.parse(pullRequestBody, this.logger);
     }
     /**
      * Given a merged pull request, build the candidate release.
@@ -253,27 +255,27 @@ class BaseStrategy {
      */
     async buildRelease(mergedPullRequest) {
         if (this.skipGitHubRelease) {
-            logger_1.logger.info('Release skipped from strategy config');
+            this.logger.info('Release skipped from strategy config');
             return;
         }
         if (!mergedPullRequest.sha) {
-            logger_1.logger.error('Pull request should have been merged');
+            this.logger.error('Pull request should have been merged');
             return;
         }
-        const pullRequestTitle = pull_request_title_1.PullRequestTitle.parse(mergedPullRequest.title, this.pullRequestTitlePattern) ||
-            pull_request_title_1.PullRequestTitle.parse(mergedPullRequest.title, manifest_1.MANIFEST_PULL_REQUEST_TITLE_PATTERN);
+        const pullRequestTitle = pull_request_title_1.PullRequestTitle.parse(mergedPullRequest.title, this.pullRequestTitlePattern, this.logger) ||
+            pull_request_title_1.PullRequestTitle.parse(mergedPullRequest.title, manifest_1.MANIFEST_PULL_REQUEST_TITLE_PATTERN, this.logger);
         if (!pullRequestTitle) {
-            logger_1.logger.error(`Bad pull request title: '${mergedPullRequest.title}'`);
+            this.logger.error(`Bad pull request title: '${mergedPullRequest.title}'`);
             return;
         }
-        const branchName = branch_name_1.BranchName.parse(mergedPullRequest.headBranchName);
+        const branchName = branch_name_1.BranchName.parse(mergedPullRequest.headBranchName, this.logger);
         if (!branchName) {
-            logger_1.logger.error(`Bad branch name: ${mergedPullRequest.headBranchName}`);
+            this.logger.error(`Bad branch name: ${mergedPullRequest.headBranchName}`);
             return;
         }
         const pullRequestBody = await this.parsePullRequestBody(mergedPullRequest.body);
         if (!pullRequestBody) {
-            logger_1.logger.error('Could not parse pull request body as a release PR');
+            this.logger.error('Could not parse pull request body as a release PR');
             return;
         }
         const component = await this.getComponent();
@@ -284,7 +286,7 @@ class BaseStrategy {
             // standalone release PR, ensure the components match
             if (this.normalizeComponent(branchName.component) !==
                 this.normalizeComponent(branchComponent)) {
-                logger_1.logger.warn(`PR component: ${branchName.component} does not match configured component: ${branchComponent}`);
+                this.logger.warn(`PR component: ${branchName.component} does not match configured component: ${branchComponent}`);
                 return;
             }
             releaseData = pullRequestBody.releaseData[0];
@@ -298,17 +300,17 @@ class BaseStrategy {
                     this.normalizeComponent(component));
             });
             if (!releaseData && pullRequestBody.releaseData.length > 0) {
-                logger_1.logger.info(`Pull request contains releases, but not for component: ${component}`);
+                this.logger.info(`Pull request contains releases, but not for component: ${component}`);
                 return;
             }
         }
         const notes = releaseData === null || releaseData === void 0 ? void 0 : releaseData.notes;
         if (notes === undefined) {
-            logger_1.logger.warn('Failed to find release notes');
+            this.logger.warn('Failed to find release notes');
         }
         const version = pullRequestTitle.getVersion() || (releaseData === null || releaseData === void 0 ? void 0 : releaseData.version);
         if (!version) {
-            logger_1.logger.error('Pull request should have included version');
+            this.logger.error('Pull request should have included version');
             return;
         }
         const tag = new tag_name_1.TagName(version, this.includeComponentInTag ? component : undefined, this.tagSeparator, this.includeVInTag);

package/build/src/strategies/dotnet-yoshi.js

@@ -17,7 +17,6 @@ exports.DotnetYoshi = void 0;
 const base_1 = require("./base");
 const changelog_1 = require("../updaters/changelog");
 const apis_1 = require("../updaters/dotnet/apis");
-const logger_1 = require("../util/logger");
 const errors_1 = require("../errors");
 const CHANGELOG_SECTIONS = [
     { type: 'feat', section: 'New features' },
@@ -77,7 +76,7 @@ class DotnetYoshi extends base_1.BaseStrategy {
         const component = await this.getComponent();
         const api = await this.getApi();
         if (api === null || api === void 0 ? void 0 : api.noVersionHistory) {
-            logger_1.logger.info(`Skipping changelog for ${component} via noVersionHistory configuration`);
+            this.logger.info(`Skipping changelog for ${component} via noVersionHistory configuration`);
         }
         else {
             updates.push({
@@ -91,7 +90,7 @@ class DotnetYoshi extends base_1.BaseStrategy {
             });
         }
         if (!component) {
-            logger_1.logger.warn('Dotnet strategy expects to use components, could not update all files');
+            this.logger.warn('Dotnet strategy expects to use components, could not update all files');
             return updates;
         }
         updates.push({