release-please 13.2.1 → 13.4.0

package/CHANGELOG.md

@@ -4,6 +4,48 @@
 
 [1]: https://www.npmjs.com/package/release-please?activeTab=versions
 
+## [13.4.0](https://github.com/googleapis/release-please/compare/v13.3.2...v13.4.0) (2022-01-18)
+
+
+### Features
+
+* add grouped manifest pr title pattern option ([#1184](https://github.com/googleapis/release-please/issues/1184)) ([df1332f](https://github.com/googleapis/release-please/commit/df1332f3b95e622752d351a29fc0a1b5083963c6))
+
+
+### Bug Fixes
+
+* **go-yoshi:** restore scope-based commit filtering ([#1233](https://github.com/googleapis/release-please/issues/1233)) ([597e6dd](https://github.com/googleapis/release-please/commit/597e6dd58217c62d562070032138cee8cad5e693))
+* search for files on target branch, not default branch ([#1235](https://github.com/googleapis/release-please/issues/1235)) ([d891f00](https://github.com/googleapis/release-please/commit/d891f000eb2b198bfb4331b41bc5020fff52e1da))
+
+### [13.3.2](https://github.com/googleapis/release-please/compare/v13.3.1...v13.3.2) (2022-01-13)
+
+
+### Bug Fixes
+
+* BranchName.parse should not throw exceptions ([#1227](https://github.com/googleapis/release-please/issues/1227)) ([364f1ac](https://github.com/googleapis/release-please/commit/364f1ac996b0120820d9bb37db96bb27a79aa936))
+* initial release version should respect Release-As commit ([#1222](https://github.com/googleapis/release-please/issues/1222)) ([22b9770](https://github.com/googleapis/release-please/commit/22b977028e2959aea088c09e3a021e2aa0e10f03))
+* Release-As commits should appear in the changelog correctly ([#1220](https://github.com/googleapis/release-please/issues/1220)) ([ab56c82](https://github.com/googleapis/release-please/commit/ab56c82e81091cbedeb8f328451ac486d7f986a4))
+* use latest Release-As commit when overriding version ([#1224](https://github.com/googleapis/release-please/issues/1224)) ([2d7cb8f](https://github.com/googleapis/release-please/commit/2d7cb8fa329d1c60b881bc0435523b81ea47a32d))
+
+### [13.3.1](https://github.com/googleapis/release-please/compare/v13.3.0...v13.3.1) (2022-01-12)
+
+
+### Bug Fixes
+
+* pass target_commitish instead of sha ([#1219](https://github.com/googleapis/release-please/issues/1219)) ([3f26ec3](https://github.com/googleapis/release-please/commit/3f26ec3de527497d7180f2cd987983c6ae6e44cd))
+
+## [13.3.0](https://github.com/googleapis/release-please/compare/v13.2.1...v13.3.0) (2022-01-12)
+
+
+### Features
+
+* allow configuring simple strategy version file ([#1168](https://github.com/googleapis/release-please/issues/1168)) ([08a0cf2](https://github.com/googleapis/release-please/commit/08a0cf2b2135713f856d8621cffa8f5b92fb8699))
+
+
+### Bug Fixes
+
+* ruby-yoshi strategy should put commit summary only in pull request body ([#1193](https://github.com/googleapis/release-please/issues/1193)) ([d29eda1](https://github.com/googleapis/release-please/commit/d29eda172597979cea5cb54ce6eb278a1dfb03a0)), closes [#1192](https://github.com/googleapis/release-please/issues/1192)
+
 ### [13.2.1](https://github.com/googleapis/release-please/compare/v13.2.0...v13.2.1) (2022-01-12)
 
 

package/build/src/changelog-notes/default.js

@@ -52,13 +52,16 @@ class DefaultChangelogNotes {
                 subject: commit.bareMessage,
                 type: commit.type,
                 scope: commit.scope,
-                notes: commit.notes,
+                notes: commit.notes.filter(note => note.title === 'BREAKING CHANGE'),
                 references: commit.references,
                 mentions: [],
                 merge: null,
                 revert: null,
                 header: commit.message,
-                footer: null,
+                footer: commit.notes
+                    .filter(note => note.title === 'RELEASE AS')
+                    .map(note => `Release-As: ${note.text}`)
+                    .join('\n'),
                 hash: commit.sha,
             };
         });

package/build/src/factory.js

@@ -76,7 +76,6 @@ const releasers = {
     'php-yoshi': options => new php_yoshi_1.PHPYoshi(options),
     python: options => new python_1.Python(options),
     rust: options => new rust_1.Rust(options),
-    simple: options => new simple_1.Simple(options),
     'terraform-module': options => new terraform_module_1.TerraformModule(options),
     helm: options => new helm_1.Helm(options),
     elixir: options => new elixir_1.Elixir(options),
@@ -159,6 +158,12 @@ async function buildStrategy(options) {
                 versioningStrategy: new service_pack_1.ServicePackVersioningStrategy(),
             });
         }
+        case 'simple': {
+            return new simple_1.Simple({
+                ...strategyOptions,
+                versionFile: options.versionFile,
+            });
+        }
         default: {
             const builder = releasers[options.releaseType];
             if (builder) {

package/build/src/github.js

@@ -304,9 +304,9 @@ class GitHub {
                 repo: this.repository.repo,
                 tag_name: release.tag.toString(),
                 body: release.notes,
-                sha: release.sha,
                 draft: !!options.draft,
                 prerelease: !!options.prerelease,
+                target_commitish: release.sha,
             });
             return {
                 name: resp.data.name || undefined,

package/build/src/manifest.d.ts

@@ -71,6 +71,7 @@ export interface ManifestOptions {
     draft?: boolean;
     prerelease?: boolean;
     draftPullRequest?: boolean;
+    groupPullRequestTitlePattern?: string;
 }
 interface ReleaserPackageConfig extends ReleaserConfigJson {
     'package-name'?: string;
@@ -88,6 +89,7 @@ export interface ManifestConfig extends ReleaserConfigJson {
     'always-link-local'?: boolean;
     plugins?: PluginType[];
     'separate-pull-requests'?: boolean;
+    'group-pull-request-title-pattern'?: string;
 }
 export declare type ReleasedVersions = Record<string, Version>;
 export declare type RepositoryConfig = Record<string, ReleaserConfig>;
@@ -122,6 +124,7 @@ export declare class Manifest {
     private draft?;
     private prerelease?;
     private draftPullRequest?;
+    private groupPullRequestTitlePattern?;
     /**
      * Create a Manifest from explicit config in code. This assumes that the
      * repository has a single component at the root path.

package/build/src/manifest.js

@@ -77,6 +77,7 @@ class Manifest {
         this.lastReleaseSha = manifestOptions === null || manifestOptions === void 0 ? void 0 : manifestOptions.lastReleaseSha;
         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;
     }
     /**
      * Create a Manifest from config files in the repository.
@@ -314,7 +315,7 @@ class Manifest {
         // Combine pull requests into 1 unless configured for separate
         // pull requests
         if (!this.separatePullRequests) {
-            plugins.push(new merge_1.Merge(this.github, this.targetBranch, this.repositoryConfig));
+            plugins.push(new merge_1.Merge(this.github, this.targetBranch, this.repositoryConfig, this.groupPullRequestTitlePattern));
         }
         for (const plugin of plugins) {
             newReleasePullRequests = await plugin.run(newReleasePullRequests);
@@ -588,6 +589,7 @@ async function parseConfig(github, configFile, branch) {
         lastReleaseSha: config['last-release-sha'],
         alwaysLinkLocal: config['always-link-local'],
         separatePullRequests: config['separate-pull-requests'],
+        groupPullRequestTitlePattern: config['group-pull-request-title-pattern'],
         plugins: config['plugins'],
     };
     return { config: repositoryConfig, options: manifestOptions };
@@ -701,7 +703,7 @@ async function latestReleaseVersion(github, targetBranch, prefix, pullRequestTit
     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;
+    var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q;
     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,
@@ -716,6 +718,7 @@ function mergeReleaserConfig(defaultConfig, pathConfig) {
         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,
+        pullRequestTitlePattern: (_q = pathConfig.pullRequestTitlePattern) !== null && _q !== void 0 ? _q : defaultConfig.pullRequestTitlePattern,
     };
 }
 /**

package/build/src/plugins/merge.d.ts

@@ -1,5 +1,6 @@
 import { ManifestPlugin } from '../plugin';
-import { CandidateReleasePullRequest } from '../manifest';
+import { CandidateReleasePullRequest, RepositoryConfig } from '../manifest';
+import { GitHub } from '../github';
 /**
  * This plugin merges multiple pull requests into a single
  * release pull request.
@@ -7,5 +8,7 @@ import { CandidateReleasePullRequest } from '../manifest';
  * Release notes are broken up using `<summary>`/`<details>` blocks.
  */
 export declare class Merge extends ManifestPlugin {
+    private pullRequestTitlePattern?;
+    constructor(github: GitHub, targetBranch: string, repositoryConfig: RepositoryConfig, pullRequestTitlePattern?: string);
     run(candidates: CandidateReleasePullRequest[]): Promise<CandidateReleasePullRequest[]>;
 }

package/build/src/plugins/merge.js

@@ -27,6 +27,11 @@ const composite_1 = require("../updaters/composite");
  * Release notes are broken up using `<summary>`/`<details>` blocks.
  */
 class Merge extends plugin_1.ManifestPlugin {
+    constructor(github, targetBranch, repositoryConfig, pullRequestTitlePattern) {
+        super(github, targetBranch, repositoryConfig);
+        this.pullRequestTitlePattern =
+            pullRequestTitlePattern || manifest_1.MANIFEST_PULL_REQUEST_TITLE_PATTERN;
+    }
     async run(candidates) {
         if (candidates.length < 1) {
             return candidates;
@@ -34,6 +39,7 @@ class Merge extends plugin_1.ManifestPlugin {
         const releaseData = [];
         const labels = new Set();
         let rawUpdates = [];
+        let rootRelease = null;
         for (const candidate of candidates) {
             const pullRequest = candidate.pullRequest;
             rawUpdates = rawUpdates.concat(...pullRequest.updates);
@@ -41,10 +47,13 @@ class Merge extends plugin_1.ManifestPlugin {
                 labels.add(label);
             }
             releaseData.push(...pullRequest.body.releaseData);
+            if (candidate.path === '.') {
+                rootRelease = candidate;
+            }
         }
         const updates = composite_1.mergeUpdates(rawUpdates);
         const pullRequest = {
-            title: pull_request_title_1.PullRequestTitle.ofTargetBranch(this.targetBranch, manifest_1.MANIFEST_PULL_REQUEST_TITLE_PATTERN),
+            title: pull_request_title_1.PullRequestTitle.ofComponentTargetBranchVersion(rootRelease === null || rootRelease === void 0 ? void 0 : rootRelease.pullRequest.title.component, this.targetBranch, rootRelease === null || rootRelease === void 0 ? void 0 : rootRelease.pullRequest.title.version, this.pullRequestTitlePattern),
             body: new pull_request_body_1.PullRequestBody(releaseData),
             updates,
             labels: Array.from(labels),

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

@@ -55,7 +55,7 @@ export declare abstract class BaseStrategy implements Strategy {
     protected tagSeparator?: string;
     private skipGitHubRelease;
     private releaseAs?;
-    private includeComponentInTag;
+    protected includeComponentInTag: boolean;
     private pullRequestTitlePattern?;
     readonly extraFiles: string[];
     readonly changelogNotes: ChangelogNotes;
@@ -79,8 +79,9 @@ export declare abstract class BaseStrategy implements Strategy {
      * @param {ConventionalCommit[]} commits parsed commits
      * @returns {ConventionalCommit[]} modified commits
      */
-    protected postProcessCommits(commits: ConventionalCommit[]): ConventionalCommit[];
+    protected postProcessCommits(commits: ConventionalCommit[]): Promise<ConventionalCommit[]>;
     protected buildReleaseNotes(conventionalCommits: ConventionalCommit[], newVersion: Version, newVersionTag: TagName, latestRelease?: Release): Promise<string>;
+    protected buildPullRequestBody(component: string | undefined, newVersion: Version, releaseNotesBody: string, _conventionalCommits: ConventionalCommit[], _latestRelease?: Release): Promise<PullRequestBody>;
     /**
      * Builds a candidate release pull request
      * @param {Commit[]} commits Raw commits to consider for this release.

package/build/src/strategies/base.js

@@ -81,7 +81,7 @@ class BaseStrategy {
      * @param {ConventionalCommit[]} commits parsed commits
      * @returns {ConventionalCommit[]} modified commits
      */
-    postProcessCommits(commits) {
+    async postProcessCommits(commits) {
         return commits;
     }
     async buildReleaseNotes(conventionalCommits, newVersion, newVersionTag, latestRelease) {
@@ -96,6 +96,15 @@ class BaseStrategy {
             changelogSections: this.changelogSections,
         });
     }
+    async buildPullRequestBody(component, newVersion, releaseNotesBody, _conventionalCommits, _latestRelease) {
+        return new pull_request_body_1.PullRequestBody([
+            {
+                component,
+                version: newVersion,
+                notes: releaseNotesBody,
+            },
+        ]);
+    }
     /**
      * Builds a candidate release pull request
      * @param {Commit[]} commits Raw commits to consider for this release.
@@ -108,13 +117,14 @@ class BaseStrategy {
      *   open a pull request.
      */
     async buildReleasePullRequest(commits, latestRelease, draft, labels = []) {
-        const conventionalCommits = this.postProcessCommits(commit_1.parseConventionalCommits(commits));
+        const conventionalCommits = await this.postProcessCommits(commit_1.parseConventionalCommits(commits));
+        logger_1.logger.info(`Considering: ${conventionalCommits.length} commits`);
         const newVersion = await this.buildNewVersion(conventionalCommits, latestRelease);
         const versionsMap = await this.updateVersionsMap(await this.buildVersionsMap(conventionalCommits), conventionalCommits);
         const component = await this.getComponent();
         logger_1.logger.debug('component:', component);
         const newVersionTag = new tag_name_1.TagName(newVersion, this.includeComponentInTag ? component : undefined, this.tagSeparator);
-        logger_1.logger.warn('pull request title pattern:', this.pullRequestTitlePattern);
+        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)
@@ -131,13 +141,7 @@ class BaseStrategy {
             latestVersion: latestRelease === null || latestRelease === void 0 ? void 0 : latestRelease.tag.version,
         });
         const updatesWithExtras = composite_1.mergeUpdates(updates.concat(...this.extraFileUpdates(newVersion)));
-        const pullRequestBody = new pull_request_body_1.PullRequestBody([
-            {
-                component,
-                version: newVersion,
-                notes: releaseNotesBody,
-            },
-        ]);
+        const pullRequestBody = await this.buildPullRequestBody(component, newVersion, releaseNotesBody, conventionalCommits, latestRelease);
         return {
             title: pullRequestTitle,
             body: pullRequestBody,
@@ -178,12 +182,17 @@ class BaseStrategy {
             logger_1.logger.warn(`Setting version for ${this.path} from release-as configuration`);
             return version_1.Version.parse(this.releaseAs);
         }
-        else if (latestRelease) {
-            return await this.versioningStrategy.bump(latestRelease.tag.version, conventionalCommits);
+        const releaseAsCommit = conventionalCommits.find(conventionalCommit => conventionalCommit.notes.find(note => note.title === 'RELEASE AS'));
+        if (releaseAsCommit) {
+            const note = releaseAsCommit.notes.find(note => note.title === 'RELEASE AS');
+            if (note) {
+                return version_1.Version.parse(note.text);
+            }
         }
-        else {
-            return this.initialReleaseVersion();
+        if (latestRelease) {
+            return await this.versioningStrategy.bump(latestRelease.tag.version, conventionalCommits);
         }
+        return this.initialReleaseVersion();
     }
     async buildVersionsMap(_conventionalCommits) {
         return new Map();

package/build/src/strategies/go-yoshi.d.ts

@@ -7,7 +7,7 @@ import { Release } from '../release';
 export declare class GoYoshi extends BaseStrategy {
     constructor(options: BaseStrategyOptions);
     protected buildUpdates(options: BuildUpdatesOptions): Promise<Update[]>;
-    protected postProcessCommits(commits: ConventionalCommit[]): ConventionalCommit[];
+    protected postProcessCommits(commits: ConventionalCommit[]): Promise<ConventionalCommit[]>;
     protected buildReleaseNotes(conventionalCommits: ConventionalCommit[], newVersion: Version, newVersionTag: TagName, latestRelease?: Release): Promise<string>;
     protected initialReleaseVersion(): Version;
 }

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

@@ -18,6 +18,7 @@ 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");
 // Commits containing a scope prefixed with an item in this array will be
 // ignored when generating a release PR for the parent module.
 const IGNORED_SUB_MODULES = new Set([
@@ -59,8 +60,10 @@ class GoYoshi extends base_1.BaseStrategy {
         });
         return updates;
     }
-    postProcessCommits(commits) {
+    async postProcessCommits(commits) {
         let regenCommit;
+        const component = await this.getComponent();
+        logger_1.logger.debug('Filtering commits');
         return commits.filter(commit => {
             var _a, _b;
             // ignore commits whose scope is in the list of ignored modules
@@ -97,6 +100,37 @@ class GoYoshi extends base_1.BaseStrategy {
                     }
                 }
             }
+            // For google-cloud-go, filter into 2 cases, a subset of modules
+            // released independently, and the remainder
+            if (this.repository.owner === 'googleapis' &&
+                this.repository.repo === 'google-cloud-go') {
+                // 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}`);
+                    return false;
+                }
+                // Skip commits related to sub-modules as they are not part of
+                // the parent module.
+                if (this.includeComponentInTag) {
+                    // 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}`);
+                        return false;
+                    }
+                }
+                else {
+                    // This is the main module release, so ignore sub modules that
+                    // are released independently
+                    for (const submodule of IGNORED_SUB_MODULES) {
+                        if (commitMatchesScope(commit.scope, submodule)) {
+                            logger_1.logger.debug(`Skipping ignored commit scope: ${commit.scope}`);
+                            return false;
+                        }
+                    }
+                }
+            }
             return true;
         });
     }
@@ -111,4 +145,7 @@ class GoYoshi extends base_1.BaseStrategy {
     }
 }
 exports.GoYoshi = GoYoshi;
+function commitMatchesScope(commitScope, scope) {
+    return commitScope === scope || commitScope.startsWith(`${scope}/`);
+}
 //# sourceMappingURL=go-yoshi.js.map

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

@@ -136,9 +136,9 @@ class JavaYoshi extends base_1.BaseStrategy {
                 versionsMap,
             }),
         });
-        const pomFilesSearch = this.github.findFilesByFilename('pom.xml', this.path);
-        const buildFilesSearch = this.github.findFilesByFilename('build.gradle', this.path);
-        const dependenciesSearch = this.github.findFilesByFilename('dependencies.properties', this.path);
+        const pomFilesSearch = this.github.findFilesByFilenameAndRef('pom.xml', this.targetBranch, this.path);
+        const buildFilesSearch = this.github.findFilesByFilenameAndRef('build.gradle', this.targetBranch, this.path);
+        const dependenciesSearch = this.github.findFilesByFilenameAndRef('dependencies.properties', this.targetBranch, this.path);
         const pomFiles = await pomFilesSearch;
         pomFiles.forEach(path => {
             updates.push({

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

@@ -50,7 +50,7 @@ class PHPYoshi extends base_1.BaseStrategy {
     }
     async buildReleasePullRequest(commits, latestRelease, draft, labels = []) {
         var _a, _b, _c;
-        const conventionalCommits = this.postProcessCommits(commit_1.parseConventionalCommits(commits));
+        const conventionalCommits = await this.postProcessCommits(commit_1.parseConventionalCommits(commits));
         const newVersion = latestRelease
             ? await this.versioningStrategy.bump(latestRelease.tag.version, conventionalCommits)
             : this.initialReleaseVersion();

package/build/src/strategies/python.js

@@ -87,7 +87,7 @@ class Python extends base_1.BaseStrategy {
             });
         }
         // There should be only one version.py, but foreach in case that is incorrect
-        const versionPyFilesSearch = this.github.findFilesByFilename('version.py', this.path);
+        const versionPyFilesSearch = this.github.findFilesByFilenameAndRef('version.py', this.targetBranch, this.path);
         const versionPyFiles = await versionPyFilesSearch;
         versionPyFiles.forEach(path => {
             updates.push({

package/build/src/strategies/ruby-yoshi.d.ts

@@ -3,7 +3,7 @@ import { ConventionalCommit } from '../commit';
 import { Update } from '../update';
 import { Release } from '../release';
 import { Version } from '../version';
-import { TagName } from '../util/tag-name';
+import { PullRequestBody } from '../util/pull-request-body';
 interface RubyYoshiStrategyOptions extends BaseStrategyOptions {
     versionFile?: string;
 }
@@ -11,7 +11,7 @@ export declare class RubyYoshi extends BaseStrategy {
     readonly versionFile: string;
     constructor(options: RubyYoshiStrategyOptions);
     protected buildUpdates(options: BuildUpdatesOptions): Promise<Update[]>;
-    protected postProcessCommits(commits: ConventionalCommit[]): ConventionalCommit[];
-    protected buildReleaseNotes(conventionalCommits: ConventionalCommit[], newVersion: Version, newVersionTag: TagName, latestRelease?: Release): Promise<string>;
+    protected postProcessCommits(commits: ConventionalCommit[]): Promise<ConventionalCommit[]>;
+    protected buildPullRequestBody(component: string | undefined, newVersion: Version, releaseNotesBody: string, conventionalCommits: ConventionalCommit[], latestRelease?: Release): Promise<PullRequestBody>;
 }
 export {};

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

@@ -24,6 +24,7 @@ const fs_1 = require("fs");
 const path_1 = require("path");
 const manifest_1 = require("../manifest");
 const logger_1 = require("../util/logger");
+const pull_request_body_1 = require("../util/pull-request-body");
 const CHANGELOG_SECTIONS = [
     { type: 'feat', section: 'Features' },
     { type: 'fix', section: 'Bug Fixes' },
@@ -73,16 +74,15 @@ class RubyYoshi extends base_1.BaseStrategy {
         });
         return updates;
     }
-    postProcessCommits(commits) {
+    async postProcessCommits(commits) {
         commits.forEach(commit => {
             commit.message = indent_commit_1.indentCommit(commit);
         });
         return commits;
     }
-    async buildReleaseNotes(conventionalCommits, newVersion, newVersionTag, latestRelease) {
-        const releaseNotes = await super.buildReleaseNotes(conventionalCommits, newVersion, newVersionTag, latestRelease);
+    async buildPullRequestBody(component, newVersion, releaseNotesBody, conventionalCommits, latestRelease) {
         if (!latestRelease) {
-            return releaseNotes;
+            return await super.buildPullRequestBody(component, newVersion, releaseNotesBody, conventionalCommits, latestRelease);
         }
         // summarize the commits that landed:
         let summary = '### Commits since last release:\n\n';
@@ -114,7 +114,15 @@ class RubyYoshi extends base_1.BaseStrategy {
             summary += `${file}\n`;
         });
         summary += `</code></pre>\n[Compare Changes](https://github.com/${repoUrl}/compare/${latestRelease.sha}...HEAD)\n`;
-        return releaseNotes + `\n---\n${summary}`;
+        return new pull_request_body_1.PullRequestBody([
+            {
+                component,
+                version: newVersion,
+                notes: releaseNotesBody,
+            },
+        ], {
+            extra: summary,
+        });
     }
 }
 exports.RubyYoshi = RubyYoshi;

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

@@ -8,6 +8,6 @@ export declare class Ruby extends BaseStrategy {
     readonly versionFile: string;
     constructor(options: RubyStrategyOptions);
     protected buildUpdates(options: BuildUpdatesOptions): Promise<Update[]>;
-    protected postProcessCommits(commits: ConventionalCommit[]): ConventionalCommit[];
+    protected postProcessCommits(commits: ConventionalCommit[]): Promise<ConventionalCommit[]>;
 }
 export {};

package/build/src/strategies/ruby.js

@@ -50,7 +50,7 @@ class Ruby extends base_1.BaseStrategy {
         });
         return updates;
     }
-    postProcessCommits(commits) {
+    async postProcessCommits(commits) {
         commits.forEach(commit => {
             commit.message = indent_commit_1.indentCommit(commit);
         });

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

@@ -1,5 +1,11 @@
-import { BaseStrategy, BuildUpdatesOptions } from './base';
+import { BaseStrategy, BuildUpdatesOptions, BaseStrategyOptions } from './base';
 import { Update } from '../update';
+interface SimpleStrategyOptions extends BaseStrategyOptions {
+    versionFile?: string;
+}
 export declare class Simple extends BaseStrategy {
+    readonly versionFile: string;
+    constructor(options: SimpleStrategyOptions);
     protected buildUpdates(options: BuildUpdatesOptions): Promise<Update[]>;
 }
+export {};

package/build/src/strategies/simple.js

@@ -20,6 +20,11 @@ const changelog_1 = require("../updaters/changelog");
 const base_1 = require("./base");
 const default_1 = require("../updaters/default");
 class Simple extends base_1.BaseStrategy {
+    constructor(options) {
+        var _a;
+        super(options);
+        this.versionFile = (_a = options.versionFile) !== null && _a !== void 0 ? _a : 'version.txt';
+    }
     async buildUpdates(options) {
         const updates = [];
         const version = options.newVersion;
@@ -32,7 +37,7 @@ class Simple extends base_1.BaseStrategy {
             }),
         });
         updates.push({
-            path: this.addPath('version.txt'),
+            path: this.addPath(this.versionFile),
             createIfMissing: false,
             updater: new default_1.DefaultUpdater({
                 version,

package/build/src/strategies/terraform-module.js

@@ -35,7 +35,7 @@ class TerraformModule extends base_1.BaseStrategy {
         });
         // Update version in README to current candidate version.
         // A module may have submodules, so find all submodules.
-        const readmeFiles = await this.github.findFilesByFilename('readme.md', this.path);
+        const readmeFiles = await this.github.findFilesByFilenameAndRef('readme.md', this.targetBranch, this.path);
         readmeFiles.forEach(path => {
             updates.push({
                 path: this.addPath(path),
@@ -48,8 +48,8 @@ class TerraformModule extends base_1.BaseStrategy {
         // Update versions.tf to current candidate version.
         // A module may have submodules, so find all versions.tfand versions.tf.tmpl to update.
         const versionFiles = await Promise.all([
-            this.github.findFilesByFilename('versions.tf', this.path),
-            this.github.findFilesByFilename('versions.tf.tmpl', this.path),
+            this.github.findFilesByFilenameAndRef('versions.tf', this.targetBranch, this.path),
+            this.github.findFilesByFilenameAndRef('versions.tf.tmpl', this.targetBranch, this.path),
         ]).then(([v, vt]) => {
             return v.concat(vt);
         });

package/build/src/util/branch-name.js

@@ -15,6 +15,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.BranchName = void 0;
 const version_1 = require("../version");
+const logger_1 = require("./logger");
 // cannot import from '..' - transpiled code references to RELEASE_PLEASE
 // at the script level are undefined, they are only defined inside function
 // or instance methods/properties.
@@ -32,13 +33,19 @@ function getAllResourceNames() {
 class BranchName {
     constructor(_branchName) { }
     static parse(branchName) {
-        const branchNameClass = getAllResourceNames().find(clazz => {
-            return clazz.matches(branchName);
-        });
-        if (!branchNameClass) {
+        try {
+            const branchNameClass = getAllResourceNames().find(clazz => {
+                return clazz.matches(branchName);
+            });
+            if (!branchNameClass) {
+                return undefined;
+            }
+            return new branchNameClass(branchName);
+        }
+        catch (e) {
+            logger_1.logger.warn(`Error parsing branch name: ${branchName}`, e);
             return undefined;
         }
-        return new branchNameClass(branchName);
     }
     static ofComponentVersion(branchPrefix, version) {
         return new AutoreleaseBranchName(`release-${branchPrefix}-v${version}`);

package/build/src/util/pull-request-body.d.ts

@@ -2,10 +2,12 @@ import { Version } from '../version';
 interface PullRequestBodyOptions {
     header?: string;
     footer?: string;
+    extra?: string;
 }
 export declare class PullRequestBody {
     header: string;
     footer: string;
+    extra?: string;
     releaseData: ReleaseData[];
     constructor(releaseData: ReleaseData[], options?: PullRequestBodyOptions);
     static parse(body: string): PullRequestBody | undefined;

package/build/src/util/pull-request-body.js

@@ -24,6 +24,7 @@ class PullRequestBody {
     constructor(releaseData, options) {
         this.header = (options === null || options === void 0 ? void 0 : options.header) || DEFAULT_HEADER;
         this.footer = (options === null || options === void 0 ? void 0 : options.footer) || DEFAULT_FOOTER;
+        this.extra = options === null || options === void 0 ? void 0 : options.extra;
         this.releaseData = releaseData;
     }
     static parse(body) {
@@ -63,7 +64,7 @@ ${NOTES_DELIMITER}
 
 ${notes}
 
-${NOTES_DELIMITER}
+${NOTES_DELIMITER}${this.extra ? `\n\n${this.extra}\n` : ''}
 ${this.footer}`;
     }
 }

package/build/src/util/pull-request-title.d.ts

@@ -11,7 +11,7 @@ export declare class PullRequestTitle {
     static ofComponentVersion(component: string, version: Version, pullRequestTitlePattern?: string): PullRequestTitle;
     static ofVersion(version: Version, pullRequestTitlePattern?: string): PullRequestTitle;
     static ofTargetBranchVersion(targetBranch: string, version: Version, pullRequestTitlePattern?: string): PullRequestTitle;
-    static ofComponentTargetBranchVersion(component: string, targetBranch: string, version: Version, pullRequestTitlePattern?: string): PullRequestTitle;
+    static ofComponentTargetBranchVersion(component?: string, targetBranch?: string, version?: Version, pullRequestTitlePattern?: string): PullRequestTitle;
     static ofTargetBranch(targetBranch: string, pullRequestTitlePattern?: string): PullRequestTitle;
     getTargetBranch(): string | undefined;
     getComponent(): string | undefined;

package/build/src/versioning-strategies/default.js

@@ -16,6 +16,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.DefaultVersioningStrategy = void 0;
 const versioning_strategy_1 = require("../versioning-strategy");
 const version_1 = require("../version");
+const logger_1 = require("../util/logger");
 /**
  * This is the default VersioningStrategy for release-please. Breaking
  * changes should bump the major, features should bump the minor, and other
@@ -48,11 +49,12 @@ class DefaultVersioningStrategy {
         // iterate through list of commits and find biggest commit type
         let breaking = 0;
         let features = 0;
-        let customVersion;
         for (const commit of commits) {
             const releaseAs = commit.notes.find(note => note.title === 'RELEASE AS');
             if (releaseAs) {
-                customVersion = version_1.Version.parse(releaseAs.text);
+                // commits are handled newest to oldest, so take the first one (newest) found
+                logger_1.logger.debug(`found Release-As: ${releaseAs.text}, forcing version`);
+                return new versioning_strategy_1.CustomVersionUpdate(version_1.Version.parse(releaseAs.text).toString());
             }
             if (commit.breaking) {
                 breaking++;
@@ -61,9 +63,6 @@ class DefaultVersioningStrategy {
                 features++;
             }
         }
-        if (customVersion) {
-            return new versioning_strategy_1.CustomVersionUpdate(customVersion.toString());
-        }
         if (breaking > 0) {
             if (version.major < 1 && this.bumpMinorPreMajor) {
                 return new versioning_strategy_1.MinorVersionUpdate();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "release-please",
-  "version": "13.2.1",
+  "version": "13.4.0",
   "description": "generate release PRs based on the conventionalcommits.org spec",
   "main": "./build/src/index.js",
   "bin": "./build/src/bin/release-please.js",
@@ -79,7 +79,7 @@
     "node-html-parser": "^5.0.0",
     "parse-github-repo-url": "^1.4.1",
     "semver": "^7.0.0",
-    "type-fest": "^1.0.0",
+    "type-fest": "^2.0.0",
     "typescript": "^3.8.3",
     "unist-util-visit": "^2.0.3",
     "unist-util-visit-parents": "^3.1.1",