release-please 13.14.0 → 13.16.0

package/CHANGELOG.md

@@ -4,6 +4,28 @@
 
 [1]: https://www.npmjs.com/package/release-please?activeTab=versions
 
+## [13.16.0](https://github.com/googleapis/release-please/compare/v13.15.1...v13.16.0) (2022-05-06)
+
+
+### Features
+
+* allow configuring separate-pull-requests per component ([#1412](https://github.com/googleapis/release-please/issues/1412)) ([d274421](https://github.com/googleapis/release-please/commit/d2744219fbbd6c58a10b177a824fb3715039162a))
+
+### [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)
 
 

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

@@ -39,9 +39,11 @@ export interface ReleaserConfig {
     includeVInTag?: boolean;
     pullRequestTitlePattern?: string;
     tagSeparator?: string;
+    separatePullRequests?: boolean;
     changelogSections?: ChangelogSection[];
     changelogPath?: string;
     changelogType?: ChangelogNotesType;
+    changelogHost?: string;
     versionFile?: string;
     extraFiles?: ExtraFile[];
     snapshotLabels?: string[];
@@ -72,7 +74,9 @@ interface ReleaserConfigJson {
     'include-component-in-tag'?: boolean;
     'include-v-in-tag'?: boolean;
     'changelog-type'?: ChangelogNotesType;
+    'changelog-host'?: string;
     'pull-request-title-pattern'?: string;
+    'separate-pull-requests'?: boolean;
     'tag-separator'?: string;
     'extra-files'?: string[];
     'version-file'?: string;
@@ -123,7 +127,6 @@ export interface ManifestConfig extends ReleaserConfigJson {
     'last-release-sha'?: string;
     'always-link-local'?: boolean;
     plugins?: PluginType[];
-    'separate-pull-requests'?: boolean;
     'group-pull-request-title-pattern'?: string;
     'release-search-depth'?: number;
     'commit-search-depth'?: number;

package/build/src/manifest.js

@@ -700,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,
@@ -714,6 +715,7 @@ function extractReleaserConfig(config) {
         changelogType: config['changelog-type'],
         pullRequestTitlePattern: config['pull-request-title-pattern'],
         tagSeparator: config['tag-separator'],
+        separatePullRequests: config['separate-pull-requests'],
     };
 }
 /**
@@ -872,25 +874,27 @@ 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, _v;
     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,
+        separatePullRequests: (_v = pathConfig.separatePullRequests) !== null && _v !== void 0 ? _v : defaultConfig.separatePullRequests,
     };
 }
 /**

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/merge.js

@@ -38,11 +38,20 @@ class Merge extends plugin_1.ManifestPlugin {
             return candidates;
         }
         logger_1.logger.info(`Merging ${candidates.length} pull requests`);
+        const [inScopeCandidates, outOfScopeCandidates] = candidates.reduce((collection, candidate) => {
+            if (candidate.config.separatePullRequests) {
+                collection[1].push(candidate);
+            }
+            else {
+                collection[0].push(candidate);
+            }
+            return collection;
+        }, [[], []]);
         const releaseData = [];
         const labels = new Set();
         let rawUpdates = [];
         let rootRelease = null;
-        for (const candidate of candidates) {
+        for (const candidate of inScopeCandidates) {
             const pullRequest = candidate.pullRequest;
             rawUpdates = rawUpdates.concat(...pullRequest.updates);
             for (const label of pullRequest.labels) {
@@ -72,6 +81,7 @@ class Merge extends plugin_1.ManifestPlugin {
                     releaseType,
                 },
             },
+            ...outOfScopeCandidates,
         ];
     }
 }

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?;
@@ -77,6 +79,7 @@ export declare abstract class BaseStrategy implements Strategy {
      */
     getComponent(): Promise<string | undefined>;
     getDefaultComponent(): Promise<string | undefined>;
+    protected getBranchComponent(): Promise<string | undefined>;
     getPackageName(): Promise<string | undefined>;
     getDefaultPackageName(): Promise<string | undefined>;
     protected normalizeComponent(component: string | undefined): string;

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;
@@ -72,6 +73,9 @@ class BaseStrategy {
         var _a;
         return this.normalizeComponent((_a = this.packageName) !== null && _a !== void 0 ? _a : (await this.getDefaultPackageName()));
     }
+    async getBranchComponent() {
+        return this.component || (await this.getDefaultComponent());
+    }
     async getPackageName() {
         var _a;
         return (_a = this.packageName) !== null && _a !== void 0 ? _a : (await this.getDefaultPackageName());
@@ -97,6 +101,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(),
@@ -141,8 +146,9 @@ class BaseStrategy {
         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);
         const pullRequestTitle = pull_request_title_1.PullRequestTitle.ofComponentTargetBranchVersion(component || '', this.targetBranch, newVersion, this.pullRequestTitlePattern);
-        const branchName = component
-            ? branch_name_1.BranchName.ofComponentTargetBranch(component, this.targetBranch)
+        const branchComponent = await this.getBranchComponent();
+        const branchName = branchComponent
+            ? branch_name_1.BranchName.ofComponentTargetBranch(branchComponent, this.targetBranch)
             : branch_name_1.BranchName.ofTargetBranch(this.targetBranch);
         const releaseNotesBody = await this.buildReleaseNotes(conventionalCommits, newVersion, newVersionTag, latestRelease, commits);
         if (this.changelogEmpty(releaseNotesBody)) {

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.14.0",
+  "version": "13.16.0",
   "description": "generate release PRs based on the conventionalcommits.org spec",
   "main": "./build/src/index.js",
   "bin": "./build/src/bin/release-please.js",