release-please 13.13.0 → 13.14.0

package/CHANGELOG.md

@@ -4,6 +4,13 @@
 
 [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)
 
 

package/build/src/manifest.d.ts

@@ -88,9 +88,10 @@ export interface ManifestOptions {
     signoff?: string;
     manifestPath?: string;
     labels?: string[];
-    skipLabeling?: boolean;
     releaseLabels?: string[];
     snapshotLabels?: string[];
+    skipLabeling?: boolean;
+    sequentialCalls?: boolean;
     draft?: boolean;
     prerelease?: boolean;
     draftPullRequest?: boolean;
@@ -126,6 +127,7 @@ export interface ManifestConfig extends ReleaserConfigJson {
     '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>;
@@ -156,6 +158,7 @@ export declare class Manifest {
     private signoffUser?;
     private labels;
     private skipLabeling?;
+    private sequentialCalls?;
     private releaseLabels;
     private snapshotLabels;
     private plugins;
@@ -247,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

@@ -79,6 +79,7 @@ class Manifest {
             (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;
@@ -400,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();
@@ -417,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');
@@ -573,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
@@ -731,6 +753,7 @@ async function parseConfig(github, configFile, branch, onlyPath, releaseAs) {
         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.13.0",
+  "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",