release-please 14.1.2 → 14.2.0

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

@@ -18,7 +18,6 @@ const base_1 = require("./base");
 const changelog_1 = require("../updaters/changelog");
 const version_1 = require("../version");
 const version_go_1 = require("../updaters/go/version-go");
-const logger_1 = require("../util/logger");
 const path_1 = require("path");
 const CHANGELOG_SECTIONS = [
     { type: 'feat', section: 'Features' },
@@ -67,7 +66,7 @@ class GoYoshi extends base_1.BaseStrategy {
     async postProcessCommits(commits) {
         let regenCommit;
         const component = await this.getComponent();
-        logger_1.logger.debug('Filtering commits');
+        this.logger.debug('Filtering commits');
         const ignoredSubmodules = await this.getIgnoredSubModules();
         return commits.filter(commit => {
             var _a, _b;
@@ -108,7 +107,7 @@ class GoYoshi extends base_1.BaseStrategy {
                 // Skip commits that don't have a scope as we don't know where to
                 // put them
                 if (!commit.scope) {
-                    logger_1.logger.debug(`Skipping commit without scope: ${commit.message}`);
+                    this.logger.debug(`Skipping commit without scope: ${commit.message}`);
                     return false;
                 }
                 // Skip commits related to sub-modules as they are not part of
@@ -117,7 +116,7 @@ class GoYoshi extends base_1.BaseStrategy {
                     // This is a submodule release, so only include commits in this
                     // scope
                     if (!commitMatchesScope(commit.scope, component)) {
-                        logger_1.logger.debug(`Skipping commit scope: ${commit.scope} != ${component}`);
+                        this.logger.debug(`Skipping commit scope: ${commit.scope} != ${component}`);
                         return false;
                     }
                 }
@@ -126,7 +125,7 @@ class GoYoshi extends base_1.BaseStrategy {
                     // are released independently
                     for (const submodule of ignoredSubmodules) {
                         if (commitMatchesScope(commit.scope, submodule)) {
-                            logger_1.logger.debug(`Skipping ignored commit scope: ${commit.scope}`);
+                            this.logger.debug(`Skipping ignored commit scope: ${commit.scope}`);
                             return false;
                         }
                     }
@@ -143,12 +142,12 @@ class GoYoshi extends base_1.BaseStrategy {
             this.includeComponentInTag) {
             return new Set();
         }
-        logger_1.logger.info('Looking for go.mod files');
+        this.logger.info('Looking for go.mod files');
         const paths = (await this.github.findFilesByFilenameAndRef('go.mod', this.targetBranch))
             .filter(path => !path.includes('internal') && path !== 'go.mod')
             .map(path => (0, path_1.dirname)(path));
-        logger_1.logger.info(`Found ${paths.length} submodules`);
-        logger_1.logger.debug(JSON.stringify(paths));
+        this.logger.info(`Found ${paths.length} submodules`);
+        this.logger.debug(JSON.stringify(paths));
         return new Set(paths);
     }
     // "closes" is a little presumptuous, let's just indicate that the

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

@@ -18,7 +18,6 @@ const versions_manifest_1 = require("../updaters/java/versions-manifest");
 const version_1 = require("../version");
 const changelog_1 = require("../updaters/changelog");
 const errors_1 = require("../errors");
-const logger_1 = require("../util/logger");
 const java_1 = require("./java");
 const java_update_1 = require("../updaters/java/java-update");
 class JavaYoshi extends java_1.Java {
@@ -162,7 +161,7 @@ class JavaYoshi extends java_1.Java {
         for (const versionKey of versionsMap.keys()) {
             const version = versionsMap.get(versionKey);
             if (!version) {
-                logger_1.logger.warn(`didn't find version for ${versionKey}`);
+                this.logger.warn(`didn't find version for ${versionKey}`);
                 continue;
             }
             if (isPromotion && isStableArtifact(versionKey)) {

package/build/src/strategies/java.js

@@ -17,7 +17,6 @@ exports.Java = void 0;
 const base_1 = require("./base");
 const changelog_1 = require("../updaters/changelog");
 const java_snapshot_1 = require("../versioning-strategies/java-snapshot");
-const logger_1 = require("../util/logger");
 const pull_request_title_1 = require("../util/pull-request-title");
 const branch_name_1 = require("../util/branch-name");
 const pull_request_body_1 = require("../util/pull-request-body");
@@ -26,6 +25,7 @@ const java_add_snapshot_1 = require("../versioning-strategies/java-add-snapshot"
 const manifest_1 = require("../manifest");
 const java_released_1 = require("../updaters/java/java-released");
 const composite_1 = require("../updaters/composite");
+const logger_1 = require("../util/logger");
 const CHANGELOG_SECTIONS = [
     { type: 'feat', section: 'Features' },
     { type: 'fix', section: 'Bug Fixes' },
@@ -47,22 +47,23 @@ const CHANGELOG_SECTIONS = [
  */
 class Java extends base_1.BaseStrategy {
     constructor(options) {
-        var _a, _b;
+        var _a, _b, _c;
         options.changelogSections = (_a = options.changelogSections) !== null && _a !== void 0 ? _a : CHANGELOG_SECTIONS;
         // wrap the configured versioning strategy with snapshotting
-        const parentVersioningStrategy = options.versioningStrategy || new default_1.DefaultVersioningStrategy();
+        const parentVersioningStrategy = options.versioningStrategy ||
+            new default_1.DefaultVersioningStrategy({ logger: (_b = options.logger) !== null && _b !== void 0 ? _b : logger_1.logger });
         options.versioningStrategy = new java_snapshot_1.JavaSnapshot(parentVersioningStrategy);
         super(options);
         this.snapshotVersioning = new java_add_snapshot_1.JavaAddSnapshot(parentVersioningStrategy);
         this.snapshotLabels = options.snapshotLabels || manifest_1.DEFAULT_SNAPSHOT_LABELS;
-        this.skipSnapshot = (_b = options.skipSnapshot) !== null && _b !== void 0 ? _b : false;
+        this.skipSnapshot = (_c = options.skipSnapshot) !== null && _c !== void 0 ? _c : false;
     }
     async buildReleasePullRequest(commits, latestRelease, draft, labels = []) {
         if (await this.needsSnapshot(commits, latestRelease)) {
-            logger_1.logger.info('Repository needs a snapshot bump.');
+            this.logger.info('Repository needs a snapshot bump.');
             return await this.buildSnapshotPullRequest(latestRelease, draft, this.snapshotLabels);
         }
-        logger_1.logger.info('No Java snapshot needed');
+        this.logger.info('No Java snapshot needed');
         return await super.buildReleasePullRequest(commits, latestRelease, draft, labels);
     }
     async buildSnapshotPullRequest(latestRelease, draft, labels = []) {
@@ -113,7 +114,7 @@ class Java extends base_1.BaseStrategy {
             return false;
         }
         const component = await this.getComponent();
-        logger_1.logger.debug('component:', component);
+        this.logger.debug('component:', component);
         const version = (_a = latestRelease === null || latestRelease === void 0 ? void 0 : latestRelease.tag) === null || _a === void 0 ? void 0 : _a.version;
         if (!version) {
             // Don't bump snapshots for the first release ever
@@ -127,7 +128,7 @@ class Java extends base_1.BaseStrategy {
         const pullRequests = commits
             .map(commit => {
             var _a;
-            return pull_request_title_1.PullRequestTitle.parse(((_a = commit.pullRequest) === null || _a === void 0 ? void 0 : _a.title) || commit.message, this.pullRequestTitlePattern);
+            return pull_request_title_1.PullRequestTitle.parse(((_a = commit.pullRequest) === null || _a === void 0 ? void 0 : _a.title) || commit.message, this.pullRequestTitlePattern, this.logger);
         })
             .filter(pullRequest => pullRequest);
         const snapshotCommits = pullRequests

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

@@ -23,7 +23,6 @@ const version_1 = require("../version");
 const commit_1 = require("../commit");
 const commit_split_1 = require("../util/commit-split");
 const default_1 = require("../updaters/default");
-const logger_1 = require("../util/logger");
 const tag_name_1 = require("../util/tag-name");
 const pull_request_title_1 = require("../util/pull-request-title");
 const branch_name_1 = require("../util/branch-name");
@@ -51,9 +50,9 @@ class PHPYoshi extends base_1.BaseStrategy {
     }
     async buildReleasePullRequest(commits, latestRelease, draft, labels = []) {
         var _a, _b, _c;
-        const conventionalCommits = await this.postProcessCommits((0, commit_1.parseConventionalCommits)(commits));
+        const conventionalCommits = await this.postProcessCommits((0, commit_1.parseConventionalCommits)(commits, this.logger));
         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 = latestRelease
@@ -115,7 +114,7 @@ class PHPYoshi extends base_1.BaseStrategy {
             const componentInfo = directoryVersionContents[directory];
             const version = versionsMap.get(componentInfo.composer.name);
             if (!version) {
-                logger_1.logger.warn(`No version found for ${componentInfo.composer.name}`);
+                this.logger.warn(`No version found for ${componentInfo.composer.name}`);
                 continue;
             }
             updates.push({
@@ -155,7 +154,7 @@ class PHPYoshi extends base_1.BaseStrategy {
         };
     }
     async parsePullRequestBody(pullRequestBody) {
-        const body = pull_request_body_1.PullRequestBody.parse(pullRequestBody);
+        const body = pull_request_body_1.PullRequestBody.parse(pullRequestBody, this.logger);
         if (!body) {
             return undefined;
         }

package/build/src/strategies/python.js

@@ -20,7 +20,6 @@ const version_1 = require("../version");
 const setup_cfg_1 = require("../updaters/python/setup-cfg");
 const setup_py_1 = require("../updaters/python/setup-py");
 const pyproject_toml_1 = require("../updaters/python/pyproject-toml");
-const logger_1 = require("../util/logger");
 const python_file_with_version_1 = require("../updaters/python/python-file-with-version");
 const CHANGELOG_SECTIONS = [
     { type: 'feat', section: 'Features' },
@@ -82,12 +81,12 @@ class Python extends base_1.BaseStrategy {
             projectName = pyProject.name;
         }
         else {
-            logger_1.logger.warn(parsedPyProject
+            this.logger.warn(parsedPyProject
                 ? 'invalid pyproject.toml'
                 : `file ${this.addPath('pyproject.toml')} did not exist`);
         }
         if (!projectName) {
-            logger_1.logger.warn('No project/component found.');
+            this.logger.warn('No project/component found.');
         }
         else {
             [projectName, projectName.replace(/-/g, '_')]

package/build/src/strategies/rust.js

@@ -20,7 +20,6 @@ const changelog_1 = require("../updaters/changelog");
 const cargo_toml_1 = require("../updaters/rust/cargo-toml");
 const cargo_lock_1 = require("../updaters/rust/cargo-lock");
 const common_1 = require("../updaters/rust/common");
-const logger_1 = require("../util/logger");
 const base_1 = require("./base");
 const version_1 = require("../version");
 class Rust extends base_1.BaseStrategy {
@@ -44,28 +43,28 @@ class Rust extends base_1.BaseStrategy {
                 versionsMap.set(workspaceManifest.package.name, version);
             }
             else {
-                logger_1.logger.warn('No workspace manifest package name found');
+                this.logger.warn('No workspace manifest package name found');
             }
-            logger_1.logger.info(`found workspace with ${members.length} members, upgrading all`);
+            this.logger.info(`found workspace with ${members.length} members, upgrading all`);
             // Collect submodule names to update
             const manifestsByPath = new Map();
             for (const member of members) {
                 const manifestPath = `${member}/Cargo.toml`;
                 const manifestContent = await this.getContent(manifestPath);
                 if (!manifestContent) {
-                    logger_1.logger.warn(`member ${member} declared but did not find Cargo.toml`);
+                    this.logger.warn(`member ${member} declared but did not find Cargo.toml`);
                     continue;
                 }
                 const manifest = (0, common_1.parseCargoManifest)(manifestContent.parsedContent);
                 manifestsByPath.set(manifestPath, manifestContent);
                 if (!((_c = manifest.package) === null || _c === void 0 ? void 0 : _c.name)) {
-                    logger_1.logger.warn(`member ${member} has no package name`);
+                    this.logger.warn(`member ${member} has no package name`);
                     continue;
                 }
                 versionsMap.set(manifest.package.name, version);
             }
-            logger_1.logger.info(`updating ${manifestsByPath.size} submodules`);
-            logger_1.logger.debug('versions map:', versionsMap);
+            this.logger.info(`updating ${manifestsByPath.size} submodules`);
+            this.logger.debug('versions map:', versionsMap);
             for (const [manifestPath, manifestContent] of manifestsByPath) {
                 updates.push({
                     path: this.addPath(manifestPath),
@@ -88,13 +87,13 @@ class Rust extends base_1.BaseStrategy {
             });
         }
         else {
-            logger_1.logger.info('single crate found, updating Cargo.toml');
+            this.logger.info('single crate found, updating Cargo.toml');
             const packageName = await this.getDefaultPackageName();
             if (packageName) {
                 versionsMap.set(packageName, version);
             }
             else {
-                logger_1.logger.warn('No crate package name found');
+                this.logger.warn('No crate package name found');
             }
             updates.push({
                 path: this.addPath('Cargo.toml'),

package/build/src/update.d.ts

@@ -1,4 +1,5 @@
 import { GitHubFileContents } from '@google-automations/git-file-utils';
+import { Logger } from './util/logger';
 /**
  * An update is a collection of data that represents changes to
  * a file in a repository.
@@ -19,5 +20,5 @@ export interface Updater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string | undefined): string;
+    updateContent(content: string | undefined, logger?: Logger): string;
 }

package/build/src/updaters/dart/pubspec-yaml.d.ts

@@ -1,3 +1,4 @@
+import { Logger } from '../../util/logger';
 import { DefaultUpdater } from '../default';
 /**
  * Updates a Dart pubspec.yaml file.
@@ -8,5 +9,5 @@ export declare class PubspecYaml extends DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string): string;
+    updateContent(content: string, logger?: Logger): string;
 }

package/build/src/updaters/dart/pubspec-yaml.js

@@ -25,7 +25,7 @@ class PubspecYaml extends default_1.DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content) {
+    updateContent(content, logger = logger_1.logger) {
         const oldVersion = content.match(/^version: ([0-9.]+)\+?(.*$)/m);
         let buildNumber = '';
         if (oldVersion) {
@@ -33,14 +33,14 @@ class PubspecYaml extends default_1.DefaultUpdater {
             const parsedBuild = parseInt(buildNumber);
             if (!isNaN(parsedBuild)) {
                 buildNumber = `+${parsedBuild + 1}`;
-                logger_1.logger.info(`updating from ${oldVersion[1]}+${oldVersion[2]} to ${this.version}${buildNumber}`);
+                logger.info(`updating from ${oldVersion[1]}+${oldVersion[2]} to ${this.version}${buildNumber}`);
             }
             else if (buildNumber.length > 0) {
                 buildNumber = `+${buildNumber}`;
-                logger_1.logger.info(`updating from ${oldVersion[1]}+${oldVersion[2]} to ${this.version}${buildNumber}`);
+                logger.info(`updating from ${oldVersion[1]}+${oldVersion[2]} to ${this.version}${buildNumber}`);
             }
             else {
-                logger_1.logger.info(`updating from ${oldVersion[1]} to ${this.version}`);
+                logger.info(`updating from ${oldVersion[1]} to ${this.version}`);
             }
         }
         return content.replace(/^version: .*$/m, `version: ${this.version}${buildNumber}`);

package/build/src/updaters/dotnet/apis.d.ts

@@ -1,5 +1,6 @@
 import { Updater } from '../../update';
 import { Version } from '../../version';
+import { Logger } from '../../util/logger';
 /**
  * Updates the apis.json format. See
  * https://github.com/googleapis/google-cloud-dotnet/blob/main/apis/README.md.
@@ -13,5 +14,5 @@ export declare class Apis implements Updater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string): string;
+    updateContent(content: string, logger?: Logger): string;
 }

package/build/src/updaters/dotnet/apis.js

@@ -30,11 +30,11 @@ class Apis {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content) {
+    updateContent(content, logger = logger_1.logger) {
         const data = JSON.parse(content);
         const api = data.apis.find(api => api.id === this.component);
         if (!api) {
-            logger_1.logger.warn(`Failed to find component: ${this.component} in apis.json`);
+            logger.warn(`Failed to find component: ${this.component} in apis.json`);
             return content;
         }
         api.version = this.version.toString();

package/build/src/updaters/elixir/elixir-mix-exs.d.ts

@@ -1,3 +1,4 @@
+import { Logger } from '../../util/logger';
 import { DefaultUpdater } from '../default';
 /**
  * Updates an Elixir mix.exs file and looks for a version string.
@@ -8,5 +9,5 @@ export declare class ElixirMixExs extends DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string): string;
+    updateContent(content: string, logger?: Logger): string;
 }

package/build/src/updaters/elixir/elixir-mix-exs.js

@@ -25,10 +25,10 @@ class ElixirMixExs extends default_1.DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content) {
+    updateContent(content, logger = logger_1.logger) {
         const oldVersion = content.match(/version: "([A-Za-z0-9_\-+.~]+)",/);
         if (oldVersion) {
-            logger_1.logger.info(`updating from ${oldVersion[1]} to ${this.version}`);
+            logger.info(`updating from ${oldVersion[1]} to ${this.version}`);
         }
         return content.replace(/version: "[A-Za-z0-9_\-+.~]+",/, `version: "${this.version}",`);
     }

package/build/src/updaters/generic-json.d.ts

@@ -1,5 +1,6 @@
 import { Updater } from '../update';
 import { Version } from '../version';
+import { Logger } from '../util/logger';
 export declare class GenericJson implements Updater {
     readonly jsonpath: string;
     readonly version: Version;
@@ -9,5 +10,5 @@ export declare class GenericJson implements Updater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string): string;
+    updateContent(content: string, logger?: Logger): string;
 }

package/build/src/updaters/generic-json.js

@@ -27,13 +27,13 @@ class GenericJson {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content) {
+    updateContent(content, logger = logger_1.logger) {
         const data = JSON.parse(content);
         const nodes = jp.apply(data, this.jsonpath, _val => {
             return this.version.toString();
         });
         if (!nodes) {
-            logger_1.logger.warn(`No entries modified in ${this.jsonpath}`);
+            logger.warn(`No entries modified in ${this.jsonpath}`);
             return content;
         }
         return (0, json_stringify_1.jsonStringify)(data, content);

package/build/src/updaters/generic-yaml.d.ts

@@ -1,5 +1,6 @@
 import { Updater } from '../update';
 import { Version } from '../version';
+import { Logger } from '../util/logger';
 /**
  * Updates YAML document according to given JSONPath.
  *
@@ -18,5 +19,5 @@ export declare class GenericYaml implements Updater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string): string;
+    updateContent(content: string, logger?: Logger): string;
 }

package/build/src/updaters/generic-yaml.js

@@ -37,14 +37,14 @@ class GenericYaml {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content) {
+    updateContent(content, logger = logger_1.logger) {
         // Parse possibly multi-document file
         let docs;
         try {
             docs = yaml.loadAll(content, null, { json: true });
         }
         catch (e) {
-            logger_1.logger.warn('Invalid yaml, cannot be parsed', e);
+            logger.warn('Invalid yaml, cannot be parsed', e);
             return content;
         }
         // Update each document
@@ -59,7 +59,7 @@ class GenericYaml {
         });
         // If nothing was modified, return original content
         if (!modified) {
-            logger_1.logger.warn(`No entries modified in ${this.jsonpath}`);
+            logger.warn(`No entries modified in ${this.jsonpath}`);
             return content;
         }
         // Stringify documents

package/build/src/updaters/generic.d.ts

@@ -1,4 +1,5 @@
 import { DefaultUpdater, UpdateOptions } from './default';
+import { Logger } from '../util/logger';
 /**
  * Options for the Generic updater.
  */
@@ -40,5 +41,5 @@ export declare class Generic extends DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string | undefined): string;
+    updateContent(content: string | undefined, logger?: Logger): string;
 }

package/build/src/updaters/generic.js

@@ -57,7 +57,7 @@ class Generic extends default_1.DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content) {
+    updateContent(content, logger = logger_1.logger) {
         if (!content) {
             return '';
         }
@@ -78,7 +78,7 @@ class Generic extends default_1.DefaultUpdater {
                     newLines.push(line.replace(VERSION_REGEX, version.toString()));
                     return;
                 default:
-                    logger_1.logger.warn(`unknown block scope: ${scope}`);
+                    logger.warn(`unknown block scope: ${scope}`);
                     newLines.push(line);
             }
         }

package/build/src/updaters/helm/chart-yaml.d.ts

@@ -1,3 +1,4 @@
+import { Logger } from '../../util/logger';
 import { DefaultUpdater } from '../default';
 /**
  * Updates a Helm chart.yaml file.
@@ -8,5 +9,5 @@ export declare class ChartYaml extends DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string): string;
+    updateContent(content: string, logger?: Logger): string;
 }

package/build/src/updaters/helm/chart-yaml.js

@@ -26,13 +26,13 @@ class ChartYaml extends default_1.DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content) {
+    updateContent(content, logger = logger_1.logger) {
         const data = yaml.load(content, { json: true });
         if (data === null || data === undefined) {
             return '';
         }
         const parsed = JSON.parse(JSON.stringify(data));
-        logger_1.logger.info(`updating from ${parsed.version} to ${this.version}`);
+        logger.info(`updating from ${parsed.version} to ${this.version}`);
         parsed.version = this.version.toString();
         return yaml.dump(parsed);
     }

package/build/src/updaters/java/java-update.d.ts

@@ -1,4 +1,5 @@
 import { DefaultUpdater, UpdateOptions } from '../default';
+import { Logger } from '../../util/logger';
 interface JavaUpdateOptions extends UpdateOptions {
     isSnapshot?: boolean;
 }
@@ -15,6 +16,6 @@ export declare class JavaUpdate extends DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string): string;
+    updateContent(content: string, logger?: Logger): string;
 }
 export {};

package/build/src/updaters/java/java-update.js

@@ -35,9 +35,9 @@ class JavaUpdate extends default_1.DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content) {
+    updateContent(content, logger = logger_1.logger) {
         if (!this.versionsMap) {
-            logger_1.logger.warn('missing versions map');
+            logger.warn('missing versions map');
             return content;
         }
         const newLines = [];

package/build/src/updaters/java/versions-manifest.d.ts

@@ -1,5 +1,6 @@
 import { JavaUpdate } from './java-update';
 import { VersionsMap } from '../../version';
+import { Logger } from '../../util/logger';
 /**
  * Updates a versions.txt file which contains current versions of
  * components within a Java repo.
@@ -11,7 +12,7 @@ export declare class VersionsManifest extends JavaUpdate {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string): string;
+    updateContent(content: string, logger?: Logger): string;
     protected updateSingleVersion(content: string, packageName: string, version: string): string;
     static parseVersions(content: string): VersionsMap;
     static needsSnapshot(content: string): boolean;

package/build/src/updaters/java/versions-manifest.js

@@ -28,9 +28,9 @@ class VersionsManifest extends java_update_1.JavaUpdate {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content) {
+    updateContent(content, logger = logger_1.logger) {
         if (!this.versionsMap) {
-            logger_1.logger.warn('missing versions map');
+            logger.warn('missing versions map');
             return content;
         }
         let newContent = content;

package/build/src/updaters/krm/krm-blueprint-version.d.ts

@@ -1,3 +1,4 @@
+import { Logger } from '../../util/logger';
 import { DefaultUpdater } from '../default';
 /**
  * Updates KMR blueprint yaml file.
@@ -8,5 +9,5 @@ export declare class KRMBlueprintVersion extends DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string): string;
+    updateContent(content: string, logger?: Logger): string;
 }

package/build/src/updaters/krm/krm-blueprint-version.js

@@ -25,7 +25,7 @@ class KRMBlueprintVersion extends default_1.DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content) {
+    updateContent(content, logger = logger_1.logger) {
         var _a;
         // js-yaml(and kpt TS SDK) does not preserve comments hence regex match
         // match starting cnrm/ ending with semver to prevent wrong updates like pinned config.kubernetes.io/function
@@ -36,7 +36,7 @@ class KRMBlueprintVersion extends default_1.DefaultUpdater {
         }
         const oldVersion = content.match(new RegExp(matchRegex));
         if (oldVersion) {
-            logger_1.logger.info(`updating from ${oldVersion[2]} to v${this.version}`);
+            logger.info(`updating from ${oldVersion[2]} to v${this.version}`);
         }
         const newVersion = content.replace(new RegExp(matchRegex, 'g'), `$1v${this.version}`);
         return newVersion;

package/build/src/updaters/node/package-json.d.ts

@@ -1,3 +1,4 @@
+import { Logger } from '../../util/logger';
 import { DefaultUpdater } from '../default';
 /**
  * This updates a Node.js package.json file's main version.
@@ -8,5 +9,5 @@ export declare class PackageJson extends DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content: string): string;
+    updateContent(content: string, logger?: Logger): string;
 }

package/build/src/updaters/node/package-json.js

@@ -26,9 +26,9 @@ class PackageJson extends default_1.DefaultUpdater {
      * @param {string} content The initial content
      * @returns {string} The updated content
      */
-    updateContent(content) {
+    updateContent(content, logger = logger_1.logger) {
         const parsed = JSON.parse(content);
-        logger_1.logger.info(`updating from ${parsed.version} to ${this.version}`);
+        logger.info(`updating from ${parsed.version} to ${this.version}`);
         parsed.version = this.version.toString();
         return (0, json_stringify_1.jsonStringify)(parsed, content);
     }

package/build/src/updaters/node/package-lock-json.d.ts

@@ -1,8 +1,9 @@
+import { Logger } from '../../util/logger';
 import { DefaultUpdater } from '../default';
 /**
  * Updates a Node.js package-lock.json file's version and '' package
  * version (for a v2 lock file).
  */
 export declare class PackageLockJson extends DefaultUpdater {
-    updateContent(content: string): string;
+    updateContent(content: string, logger?: Logger): string;
 }