release-please 13.13.0 → 13.15.1

package/CHANGELOG.md

@@ -4,6 +4,28 @@
 
 [1]: https://www.npmjs.com/package/release-please?activeTab=versions
 
+### [13.15.1](https://github.com/googleapis/release-please/compare/v13.15.0...v13.15.1) (2022-05-05)
+
+
+### Bug Fixes
+
+* **cargo-workspace:** stop defaulting to updating all components ([#1414](https://github.com/googleapis/release-please/issues/1414)) ([532637c](https://github.com/googleapis/release-please/commit/532637c399d38962e1c0a8622656d61e0fe7e405))
+* node-workspace should not bump versions for peer dependencies ([#1413](https://github.com/googleapis/release-please/issues/1413)) ([cc4eaaa](https://github.com/googleapis/release-please/commit/cc4eaaa864b7a6444e4dc72d1c75a596932d7b62)), closes [#1403](https://github.com/googleapis/release-please/issues/1403)
+
+## [13.15.0](https://github.com/googleapis/release-please/compare/v13.14.0...v13.15.0) (2022-04-27)
+
+
+### Features
+
+* allow configuring changelog-host ([#1408](https://github.com/googleapis/release-please/issues/1408)) ([d7d525f](https://github.com/googleapis/release-please/commit/d7d525f283f931dd999ca69228e71dd6adf9e0c3))
+
+## [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/bin/release-please.d.ts

@@ -55,6 +55,7 @@ interface PullRequestStrategyArgs {
     snapshot?: boolean;
     changelogSections?: ChangelogSection[];
     changelogPath?: string;
+    changelogHost?: string;
     versioningStrategy?: VersioningStrategyType;
     versionFile?: string;
     extraFiles?: string[];

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

@@ -185,6 +185,10 @@ function pullRequestStrategyOptions(yargs) {
             }
             return arg;
         },
+    })
+        .option('changelog-host', {
+        describe: 'host for hyperlinks in the changelog',
+        type: 'string',
     })
         .option('last-package-version', {
         describe: 'last version # that package was released as',
@@ -284,6 +288,7 @@ const createReleasePullRequestCommand = {
                 bumpPatchForMinorPreMajor: argv.bumpPatchForMinorPreMajor,
                 changelogPath: argv.changelogPath,
                 changelogType: argv.changelogType,
+                changelogHost: argv.changelogHost,
                 pullRequestTitlePattern: argv.pullRequestTitlePattern,
                 changelogSections: argv.changelogSections,
                 releaseAs: argv.releaseAs,
@@ -471,6 +476,7 @@ const bootstrapCommand = {
             bumpMinorPreMajor: argv.bumpMinorPreMajor,
             bumpPatchForMinorPreMajor: argv.bumpPatchForMinorPreMajor,
             changelogPath: argv.changelogPath,
+            changelogHost: argv.changelogHost,
             changelogSections: argv.changelogSections,
             releaseAs: argv.releaseAs,
             versioning: argv.versioningStrategy,

package/build/src/manifest.d.ts

@@ -42,6 +42,7 @@ export interface ReleaserConfig {
     changelogSections?: ChangelogSection[];
     changelogPath?: string;
     changelogType?: ChangelogNotesType;
+    changelogHost?: string;
     versionFile?: string;
     extraFiles?: ExtraFile[];
     snapshotLabels?: string[];
@@ -72,6 +73,7 @@ interface ReleaserConfigJson {
     'include-component-in-tag'?: boolean;
     'include-v-in-tag'?: boolean;
     'changelog-type'?: ChangelogNotesType;
+    'changelog-host'?: string;
     'pull-request-title-pattern'?: string;
     'tag-separator'?: string;
     'extra-files'?: string[];
@@ -88,9 +90,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 +129,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 +160,7 @@ export declare class Manifest {
     private signoffUser?;
     private labels;
     private skipLabeling?;
+    private sequentialCalls?;
     private releaseLabels;
     private snapshotLabels;
     private plugins;
@@ -247,7 +252,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
@@ -678,6 +700,7 @@ function extractReleaserConfig(config) {
         bumpPatchForMinorPreMajor: config['bump-patch-for-minor-pre-major'],
         changelogSections: config['changelog-sections'],
         changelogPath: config['changelog-path'],
+        changelogHost: config['changelog-host'],
         releaseAs: config['release-as'],
         skipGithubRelease: config['skip-github-release'],
         draft: config.draft,
@@ -731,6 +754,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 };
 }
@@ -849,25 +873,26 @@ async function latestReleaseVersion(github, targetBranch, releaseFilter, prefix,
     return candidateTagVersion.sort((a, b) => b.compare(a))[0];
 }
 function mergeReleaserConfig(defaultConfig, pathConfig) {
-    var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s, _t;
+    var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s, _t, _u;
     return {
         releaseType: (_b = (_a = pathConfig.releaseType) !== null && _a !== void 0 ? _a : defaultConfig.releaseType) !== null && _b !== void 0 ? _b : 'node',
         bumpMinorPreMajor: (_c = pathConfig.bumpMinorPreMajor) !== null && _c !== void 0 ? _c : defaultConfig.bumpMinorPreMajor,
         bumpPatchForMinorPreMajor: (_d = pathConfig.bumpPatchForMinorPreMajor) !== null && _d !== void 0 ? _d : defaultConfig.bumpPatchForMinorPreMajor,
         changelogSections: (_e = pathConfig.changelogSections) !== null && _e !== void 0 ? _e : defaultConfig.changelogSections,
         changelogPath: (_f = pathConfig.changelogPath) !== null && _f !== void 0 ? _f : defaultConfig.changelogPath,
-        releaseAs: (_g = pathConfig.releaseAs) !== null && _g !== void 0 ? _g : defaultConfig.releaseAs,
-        skipGithubRelease: (_h = pathConfig.skipGithubRelease) !== null && _h !== void 0 ? _h : defaultConfig.skipGithubRelease,
-        draft: (_j = pathConfig.draft) !== null && _j !== void 0 ? _j : defaultConfig.draft,
-        prerelease: (_k = pathConfig.prerelease) !== null && _k !== void 0 ? _k : defaultConfig.prerelease,
-        component: (_l = pathConfig.component) !== null && _l !== void 0 ? _l : defaultConfig.component,
-        packageName: (_m = pathConfig.packageName) !== null && _m !== void 0 ? _m : defaultConfig.packageName,
-        versionFile: (_o = pathConfig.versionFile) !== null && _o !== void 0 ? _o : defaultConfig.versionFile,
-        extraFiles: (_p = pathConfig.extraFiles) !== null && _p !== void 0 ? _p : defaultConfig.extraFiles,
-        includeComponentInTag: (_q = pathConfig.includeComponentInTag) !== null && _q !== void 0 ? _q : defaultConfig.includeComponentInTag,
-        includeVInTag: (_r = pathConfig.includeVInTag) !== null && _r !== void 0 ? _r : defaultConfig.includeVInTag,
-        tagSeparator: (_s = pathConfig.tagSeparator) !== null && _s !== void 0 ? _s : defaultConfig.tagSeparator,
-        pullRequestTitlePattern: (_t = pathConfig.pullRequestTitlePattern) !== null && _t !== void 0 ? _t : defaultConfig.pullRequestTitlePattern,
+        changelogHost: (_g = pathConfig.changelogHost) !== null && _g !== void 0 ? _g : defaultConfig.changelogHost,
+        releaseAs: (_h = pathConfig.releaseAs) !== null && _h !== void 0 ? _h : defaultConfig.releaseAs,
+        skipGithubRelease: (_j = pathConfig.skipGithubRelease) !== null && _j !== void 0 ? _j : defaultConfig.skipGithubRelease,
+        draft: (_k = pathConfig.draft) !== null && _k !== void 0 ? _k : defaultConfig.draft,
+        prerelease: (_l = pathConfig.prerelease) !== null && _l !== void 0 ? _l : defaultConfig.prerelease,
+        component: (_m = pathConfig.component) !== null && _m !== void 0 ? _m : defaultConfig.component,
+        packageName: (_o = pathConfig.packageName) !== null && _o !== void 0 ? _o : defaultConfig.packageName,
+        versionFile: (_p = pathConfig.versionFile) !== null && _p !== void 0 ? _p : defaultConfig.versionFile,
+        extraFiles: (_q = pathConfig.extraFiles) !== null && _q !== void 0 ? _q : defaultConfig.extraFiles,
+        includeComponentInTag: (_r = pathConfig.includeComponentInTag) !== null && _r !== void 0 ? _r : defaultConfig.includeComponentInTag,
+        includeVInTag: (_s = pathConfig.includeVInTag) !== null && _s !== void 0 ? _s : defaultConfig.includeVInTag,
+        tagSeparator: (_t = pathConfig.tagSeparator) !== null && _t !== void 0 ? _t : defaultConfig.tagSeparator,
+        pullRequestTitlePattern: (_u = pathConfig.pullRequestTitlePattern) !== null && _u !== void 0 ? _u : defaultConfig.pullRequestTitlePattern,
     };
 }
 /**

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

@@ -1,8 +1,7 @@
-import { CandidateReleasePullRequest, RepositoryConfig } from '../manifest';
-import { WorkspacePlugin, DependencyGraph, WorkspacePluginOptions } from './workspace';
+import { CandidateReleasePullRequest } from '../manifest';
+import { WorkspacePlugin, DependencyGraph } from './workspace';
 import { CargoManifest } from '../updaters/rust/common';
 import { VersionsMap, Version } from '../version';
-import { GitHub } from '../github';
 interface CrateInfo {
     /**
      * e.g. `crates/crate-a`
@@ -37,7 +36,6 @@ interface CrateInfo {
  * into a single rust package.
  */
 export declare class CargoWorkspace extends WorkspacePlugin<CrateInfo> {
-    constructor(github: GitHub, targetBranch: string, repositoryConfig: RepositoryConfig, options?: WorkspacePluginOptions);
     protected buildAllPackages(candidates: CandidateReleasePullRequest[]): Promise<{
         allPackages: CrateInfo[];
         candidatesByPackage: Record<string, CandidateReleasePullRequest>;

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

@@ -35,12 +35,6 @@ const cargo_lock_1 = require("../updaters/rust/cargo-lock");
  * into a single rust package.
  */
 class CargoWorkspace extends workspace_1.WorkspacePlugin {
-    constructor(github, targetBranch, repositoryConfig, options = {}) {
-        super(github, targetBranch, repositoryConfig, {
-            ...options,
-            updateAllPackages: true,
-        });
-    }
     async buildAllPackages(candidates) {
         var _a, _b, _c, _d;
         const cargoManifestContent = await this.github.getFileContentsOnBranch('Cargo.toml', this.targetBranch);

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

@@ -211,7 +211,7 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
         return candidates;
     }
     async buildGraph(allPackages) {
-        var _a, _b, _c, _d;
+        var _a, _b, _c;
         const graph = new Map();
         const workspacePackageNames = new Set(allPackages.map(packageJson => packageJson.name));
         for (const packageJson of allPackages) {
@@ -219,7 +219,6 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
                 ...((_a = packageJson.dependencies) !== null && _a !== void 0 ? _a : {}),
                 ...((_b = packageJson.devDependencies) !== null && _b !== void 0 ? _b : {}),
                 ...((_c = packageJson.optionalDependencies) !== null && _c !== void 0 ? _c : {}),
-                ...((_d = packageJson.peerDependencies) !== null && _d !== void 0 ? _d : {}),
             });
             const workspaceDeps = allDeps.filter(dep => workspacePackageNames.has(dep));
             graph.set(packageJson.name, {

package/build/src/plugins/workspace.js

@@ -132,6 +132,7 @@ class WorkspacePlugin extends plugin_1.ManifestPlugin {
      *   being updated.
      */
     buildGraphOrder(graph, packageNamesToUpdate) {
+        logger_1.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();

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

@@ -28,6 +28,7 @@ export interface BaseStrategyOptions {
     versioningStrategy?: VersioningStrategy;
     targetBranch: string;
     changelogPath?: string;
+    changelogHost?: string;
     changelogSections?: ChangelogSection[];
     commitPartial?: string;
     headerPartial?: string;
@@ -56,6 +57,7 @@ export declare abstract class BaseStrategy implements Strategy {
     protected targetBranch: string;
     protected repository: Repository;
     protected changelogPath: string;
+    protected changelogHost?: string;
     protected tagSeparator?: string;
     private skipGitHubRelease;
     private releaseAs?;

package/build/src/strategies/base.js

@@ -47,6 +47,7 @@ class BaseStrategy {
         this.targetBranch = options.targetBranch;
         this.repository = options.github.repository;
         this.changelogPath = options.changelogPath || DEFAULT_CHANGELOG_PATH;
+        this.changelogHost = options.changelogHost;
         this.changelogSections = options.changelogSections;
         this.tagSeparator = options.tagSeparator;
         this.skipGitHubRelease = options.skipGitHubRelease || false;
@@ -97,6 +98,7 @@ class BaseStrategy {
     async buildReleaseNotes(conventionalCommits, newVersion, newVersionTag, latestRelease, commits) {
         var _a;
         return await this.changelogNotes.buildNotes(conventionalCommits, {
+            host: this.changelogHost,
             owner: this.repository.owner,
             repository: this.repository.repo,
             version: newVersion.toString(),

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

@@ -79,6 +79,7 @@ class PHPYoshi extends base_1.BaseStrategy {
                 const newVersion = await this.versioningStrategy.bump(version, splitCommits[directory]);
                 versionsMap.set(composer.name, newVersion);
                 const partialReleaseNotes = await this.changelogNotes.buildNotes(splitCommits[directory], {
+                    host: this.changelogHost,
                     owner: this.repository.owner,
                     repository: this.repository.repo,
                     version: newVersion.toString(),

package/package.json

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