release-please 13.11.1 → 13.14.0

package/CHANGELOG.md

@@ -4,6 +4,32 @@
 
 [1]: https://www.npmjs.com/package/release-please?activeTab=versions
 
+## [13.14.0](https://github.com/googleapis/release-please/compare/v13.13.0...v13.14.0) (2022-04-20)
+
+
+### Features
+
+* Support sequential-calls manifest field that disables concurrency when creating multiple pull requests or releases ([#1401](https://github.com/googleapis/release-please/issues/1401)) ([50f5c99](https://github.com/googleapis/release-please/commit/50f5c990b99d991b874ba88556386c6b940743f6))
+
+## [13.13.0](https://github.com/googleapis/release-please/compare/v13.12.0...v13.13.0) (2022-04-18)
+
+
+### Features
+
+* add a flag to disable adding labels to new pull requests ([#1399](https://github.com/googleapis/release-please/issues/1399)) ([3957ef5](https://github.com/googleapis/release-please/commit/3957ef542512eb1ae2c3353b3c2a7fde4540c731))
+
+## [13.12.0](https://github.com/googleapis/release-please/compare/v13.11.1...v13.12.0) (2022-04-15)
+
+
+### Features
+
+* allow configuring `release-search-depth` and `commit-search-depth` ([#1396](https://github.com/googleapis/release-please/issues/1396)) ([102d650](https://github.com/googleapis/release-please/commit/102d650394140667d17d84726bd962477d69562c)), closes [#1394](https://github.com/googleapis/release-please/issues/1394)
+
+
+### Bug Fixes
+
+* Allow an empty value for --label ([#1397](https://github.com/googleapis/release-please/issues/1397)) ([f5aff97](https://github.com/googleapis/release-please/commit/f5aff97dbb20086a26b846dde89b289a4540dba1))
+
 ### [13.11.1](https://github.com/googleapis/release-please/compare/v13.11.0...v13.11.1) (2022-04-15)
 
 

package/build/src/bin/release-please.d.ts

@@ -48,6 +48,7 @@ interface ReleaseArgs {
 interface PullRequestArgs {
     draftPullRequest?: boolean;
     label?: string;
+    skipLabeling?: boolean;
     signoff?: string;
 }
 interface PullRequestStrategyArgs {

package/build/src/bin/release-please.js

@@ -105,6 +105,11 @@ function pullRequestOptions(yargs) {
         .option('label', {
         default: 'autorelease: pending',
         describe: 'comma-separated list of labels to add to from release PR',
+    })
+        .option('skip-labeling', {
+        describe: 'skip application of labels to pull requests',
+        type: 'boolean',
+        default: false,
     })
         .option('fork', {
         describe: 'should the PR be created from a fork',
@@ -516,8 +521,14 @@ function extractManifestOptions(argv) {
     if ('fork' in argv && argv.fork !== undefined) {
         manifestOptions.fork = argv.fork;
     }
-    if (argv.label) {
-        manifestOptions.labels = argv.label.split(',');
+    if (argv.label !== undefined) {
+        let labels = argv.label.split(',');
+        if (labels.length === 1 && labels[0] === '')
+            labels = [];
+        manifestOptions.labels = labels;
+    }
+    if ('skipLabeling' in argv && argv.skipLabeling !== undefined) {
+        manifestOptions.skipLabeling = argv.skipLabeling;
     }
     if ('releaseLabel' in argv && argv.releaseLabel) {
         manifestOptions.releaseLabels = argv.releaseLabel.split(',');

package/build/src/github.d.ts

@@ -233,6 +233,7 @@ export declare class GitHub {
     createReleasePullRequest(releasePullRequest: ReleasePullRequest, targetBranch: string, options?: {
         signoffUser?: string;
         fork?: boolean;
+        skipLabeling?: boolean;
     }): Promise<PullRequest>;
     createPullRequest: (pullRequest: PullRequest, targetBranch: string, message: string, updates: Update[], options?: {
         fork?: boolean | undefined;

package/build/src/github.js

@@ -870,13 +870,15 @@ class GitHub {
         if (options === null || options === void 0 ? void 0 : options.signoffUser) {
             message = signoff_commit_message_1.signoffCommitMessage(message, options.signoffUser);
         }
+        const pullRequestLabels = (options === null || options === void 0 ? void 0 : options.skipLabeling) ? []
+            : releasePullRequest.labels;
         return await this.createPullRequest({
             headBranchName: releasePullRequest.headRefName,
             baseBranchName: targetBranch,
             number: -1,
             title: releasePullRequest.title.toString(),
             body: releasePullRequest.body.toString().slice(0, MAX_ISSUE_BODY_SIZE),
-            labels: releasePullRequest.labels,
+            labels: pullRequestLabels,
             files: [],
         }, targetBranch, message, releasePullRequest.updates, {
             fork: options === null || options === void 0 ? void 0 : options.fork,

package/build/src/manifest.d.ts

@@ -90,10 +90,14 @@ export interface ManifestOptions {
     labels?: string[];
     releaseLabels?: string[];
     snapshotLabels?: string[];
+    skipLabeling?: boolean;
+    sequentialCalls?: boolean;
     draft?: boolean;
     prerelease?: boolean;
     draftPullRequest?: boolean;
     groupPullRequestTitlePattern?: string;
+    releaseSearchDepth?: number;
+    commitSearchDepth?: number;
 }
 interface ReleaserPackageConfig extends ReleaserConfigJson {
     'package-name'?: string;
@@ -121,6 +125,9 @@ export interface ManifestConfig extends ReleaserConfigJson {
     plugins?: PluginType[];
     'separate-pull-requests'?: boolean;
     'group-pull-request-title-pattern'?: string;
+    'release-search-depth'?: number;
+    'commit-search-depth'?: number;
+    'sequential-calls'?: boolean;
 }
 export declare type ReleasedVersions = Record<string, Version>;
 export declare type RepositoryConfig = Record<string, ReleaserConfig>;
@@ -150,6 +157,8 @@ export declare class Manifest {
     readonly fork: boolean;
     private signoffUser?;
     private labels;
+    private skipLabeling?;
+    private sequentialCalls?;
     private releaseLabels;
     private snapshotLabels;
     private plugins;
@@ -162,6 +171,8 @@ export declare class Manifest {
     private prerelease?;
     private draftPullRequest?;
     private groupPullRequestTitlePattern?;
+    readonly releaseSearchDepth: number;
+    readonly commitSearchDepth: number;
     /**
      * Create a Manifest from explicit config in code. This assumes that the
      * repository has a single component at the root path.
@@ -239,7 +250,7 @@ export declare class Manifest {
     /**
      * Opens/updates all candidate release pull requests for this repository.
      *
-     * @returns {number[]} Pull request numbers of release pull requests
+     * @returns {PullRequest[]} Pull request numbers of release pull requests
      */
     createPullRequests(): Promise<(PullRequest | undefined)[]>;
     private findOpenReleasePullRequests;

package/build/src/manifest.js

@@ -33,6 +33,8 @@ exports.DEFAULT_LABELS = ['autorelease: pending'];
 exports.DEFAULT_RELEASE_LABELS = ['autorelease: tagged'];
 exports.DEFAULT_SNAPSHOT_LABELS = ['autorelease: snapshot'];
 exports.SNOOZE_LABEL = 'autorelease: snooze';
+const DEFAULT_RELEASE_SEARCH_DEPTH = 400;
+const DEFAULT_COMMIT_SEARCH_DEPTH = 500;
 exports.MANIFEST_PULL_REQUEST_TITLE_PATTERN = 'chore: release ${branch}';
 class Manifest {
     /**
@@ -76,6 +78,8 @@ class Manifest {
         this.releaseLabels =
             (manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.releaseLabels) || exports.DEFAULT_RELEASE_LABELS;
         this.labels = (manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.labels) || exports.DEFAULT_LABELS;
+        this.skipLabeling = (manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.skipLabeling) || false;
+        this.sequentialCalls = (manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.sequentialCalls) || false;
         this.snapshotLabels =
             (manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.snapshotLabels) || exports.DEFAULT_SNAPSHOT_LABELS;
         this.bootstrapSha = manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.bootstrapSha;
@@ -83,6 +87,10 @@ class Manifest {
         this.draft = manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.draft;
         this.draftPullRequest = manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.draftPullRequest;
         this.groupPullRequestTitlePattern = manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.groupPullRequestTitlePattern;
+        this.releaseSearchDepth =
+            (manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.releaseSearchDepth) || DEFAULT_RELEASE_SEARCH_DEPTH;
+        this.commitSearchDepth =
+            (manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.commitSearchDepth) || DEFAULT_COMMIT_SEARCH_DEPTH;
     }
     /**
      * Create a Manifest from config files in the repository.
@@ -169,8 +177,9 @@ class Manifest {
         const releaseShasByPath = {};
         // Releases by path
         const releasesByPath = {};
+        logger_1.logger.debug(`release search depth: ${this.releaseSearchDepth}`);
         for await (const release of this.github.releaseIterator({
-            maxResults: 400,
+            maxResults: this.releaseSearchDepth,
         })) {
             const tagName = tag_name_1.TagName.parse(release.tagName);
             if (!tagName) {
@@ -227,8 +236,9 @@ class Manifest {
         // seen all release commits
         logger_1.logger.info('Collecting commits since all latest releases');
         const commits = [];
+        logger_1.logger.debug(`commit search depth: ${this.commitSearchDepth}`);
         const commitGenerator = this.github.mergeCommitIterator(this.targetBranch, {
-            maxResults: 500,
+            maxResults: this.commitSearchDepth,
             backfillFiles: true,
         });
         const releaseShas = new Set(Object.values(releaseShasByPath));
@@ -391,7 +401,7 @@ class Manifest {
     /**
      * Opens/updates all candidate release pull requests for this repository.
      *
-     * @returns {number[]} Pull request numbers of release pull requests
+     * @returns {PullRequest[]} Pull request numbers of release pull requests
      */
     async createPullRequests() {
         const candidatePullRequests = await this.buildPullRequests();
@@ -408,13 +418,24 @@ class Manifest {
         // collect open and snoozed release pull requests
         const openPullRequests = await this.findOpenReleasePullRequests();
         const snoozedPullRequests = await this.findSnoozedReleasePullRequests();
-        const promises = [];
-        for (const pullRequest of candidatePullRequests) {
-            promises.push(this.createOrUpdatePullRequest(pullRequest, openPullRequests, snoozedPullRequests));
+        if (this.sequentialCalls) {
+            const pullRequests = [];
+            for (const pullRequest of candidatePullRequests) {
+                const resultPullRequest = await this.createOrUpdatePullRequest(pullRequest, openPullRequests, snoozedPullRequests);
+                if (resultPullRequest)
+                    pullRequests.push(resultPullRequest);
+            }
+            return pullRequests;
+        }
+        else {
+            const promises = [];
+            for (const pullRequest of candidatePullRequests) {
+                promises.push(this.createOrUpdatePullRequest(pullRequest, openPullRequests, snoozedPullRequests));
+            }
+            const pullNumbers = await Promise.all(promises);
+            // reject any pull numbers that were not created or updated
+            return pullNumbers.filter(number => !!number);
         }
-        const pullNumbers = await Promise.all(promises);
-        // reject any pull numbers that were not created or updated
-        return pullNumbers.filter(number => !!number);
     }
     async findOpenReleasePullRequests() {
         logger_1.logger.info('Looking for open release pull requests');
@@ -459,6 +480,7 @@ class Manifest {
         const newPullRequest = await this.github.createReleasePullRequest(pullRequest, this.targetBranch, {
             fork: this.fork,
             signoffUser: this.signoffUser,
+            skipLabeling: this.skipLabeling,
         });
         return newPullRequest;
     }
@@ -563,12 +585,22 @@ class Manifest {
                 releasesByPullRequest[release.pullRequest.number] = [release];
             }
         }
-        const promises = [];
-        for (const pullNumber in releasesByPullRequest) {
-            promises.push(this.createReleasesForPullRequest(releasesByPullRequest[pullNumber], pullRequestsByNumber[pullNumber]));
+        if (this.sequentialCalls) {
+            const resultReleases = [];
+            for (const pullNumber in releasesByPullRequest) {
+                const releases = await this.createReleasesForPullRequest(releasesByPullRequest[pullNumber], pullRequestsByNumber[pullNumber]);
+                resultReleases.concat(releases);
+            }
+            return resultReleases;
+        }
+        else {
+            const promises = [];
+            for (const pullNumber in releasesByPullRequest) {
+                promises.push(this.createReleasesForPullRequest(releasesByPullRequest[pullNumber], pullRequestsByNumber[pullNumber]));
+            }
+            const releases = await Promise.all(promises);
+            return releases.reduce((collection, r) => collection.concat(r), []);
         }
-        const releases = await Promise.all(promises);
-        return releases.reduce((collection, r) => collection.concat(r), []);
     }
     async createReleasesForPullRequest(releases, pullRequest) {
         // create the release
@@ -719,6 +751,9 @@ async function parseConfig(github, configFile, branch, onlyPath, releaseAs) {
         labels: configLabel === undefined ? undefined : [configLabel],
         releaseLabels: configReleaseLabel === undefined ? undefined : [configReleaseLabel],
         snapshotLabels: configSnapshotLabel === undefined ? undefined : [configSnapshotLabel],
+        releaseSearchDepth: config['release-search-depth'],
+        commitSearchDepth: config['commit-search-depth'],
+        sequentialCalls: config['sequential-calls'],
     };
     return { config: repositoryConfig, options: manifestOptions };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "release-please",
-  "version": "13.11.1",
+  "version": "13.14.0",
   "description": "generate release PRs based on the conventionalcommits.org spec",
   "main": "./build/src/index.js",
   "bin": "./build/src/bin/release-please.js",