release-please 13.4.13 → 13.5.0

package/CHANGELOG.md

@@ -4,6 +4,32 @@
 
 [1]: https://www.npmjs.com/package/release-please?activeTab=versions
 
+## [13.5.0](https://github.com/googleapis/release-please/compare/v13.4.15...v13.5.0) (2022-03-08)
+
+
+### Features
+
+* **ocaml:** update opam `project.opam.locked` files ([#1325](https://github.com/googleapis/release-please/issues/1325)) ([858199e](https://github.com/googleapis/release-please/commit/858199e4fac71ce2574c4195b85dc4e8bf26e0f7))
+
+
+### Bug Fixes
+
+* fix regex for parsing major version update ([#1330](https://github.com/googleapis/release-please/issues/1330)) ([afadec9](https://github.com/googleapis/release-please/commit/afadec95aa62aa32372512455326c4cdf8943f70))
+
+### [13.4.15](https://github.com/googleapis/release-please/compare/v13.4.14...v13.4.15) (2022-03-02)
+
+
+### Bug Fixes
+
+* tagging many GitHub releases should not require a file touched check ([#1321](https://github.com/googleapis/release-please/issues/1321)) ([ab99242](https://github.com/googleapis/release-please/commit/ab9924215e68838e6ec62d283b5712cbff6e6a4c))
+
+### [13.4.14](https://github.com/googleapis/release-please/compare/v13.4.13...v13.4.14) (2022-03-01)
+
+
+### Bug Fixes
+
+* node-workspace plugin should update package.json versions ([#1319](https://github.com/googleapis/release-please/issues/1319)) ([e2aaacb](https://github.com/googleapis/release-please/commit/e2aaacbab59c7660abe572c5e6ce31ad666a90c5))
+
 ### [13.4.13](https://github.com/googleapis/release-please/compare/v13.4.12...v13.4.13) (2022-02-28)
 
 

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

@@ -332,7 +332,18 @@ const createReleaseCommand = {
         }
         if (argv.dryRun) {
             const releases = await manifest.buildReleases();
-            logger_1.logger.info(releases);
+            logger_1.logger.info(`Would tag ${releases.length} releases:`);
+            for (const release of releases) {
+                logger_1.logger.info({
+                    name: release.name,
+                    tag: release.tag.toString(),
+                    notes: release.notes,
+                    sha: release.sha,
+                    draft: release.draft,
+                    prerelease: release.prerelease,
+                    pullNumber: release.pullRequest.number,
+                });
+            }
         }
         else {
             const releaseNumbers = await manifest.createReleases();

package/build/src/manifest.js

@@ -454,29 +454,11 @@ class Manifest {
         const generator = await this.findMergedReleasePullRequests();
         const releases = [];
         for await (const pullRequest of generator) {
-            logger_1.logger.info('Looking at files touched by path');
-            const cs = new commit_split_1.CommitSplit({
-                includeEmpty: true,
-                packagePaths: Object.keys(this.repositoryConfig),
-            });
-            const commits = [
-                {
-                    sha: pullRequest.sha,
-                    message: pullRequest.title,
-                    files: pullRequest.files,
-                },
-            ];
-            const commitsPerPath = cs.split(commits);
             for (const path in this.repositoryConfig) {
                 const config = this.repositoryConfig[path];
                 logger_1.logger.info(`Building release for path: ${path}`);
                 logger_1.logger.debug(`type: ${config.releaseType}`);
                 logger_1.logger.debug(`targetBranch: ${this.targetBranch}`);
-                const pathCommits = path === exports.ROOT_PROJECT_PATH ? commits : commitsPerPath[path];
-                if (!pathCommits || pathCommits.length === 0) {
-                    logger_1.logger.info(`No commits for path: ${path}, skipping`);
-                    continue;
-                }
                 const strategy = strategiesByPath[path];
                 const release = await strategy.buildRelease(pullRequest);
                 if (release) {
@@ -490,6 +472,9 @@ class Manifest {
                                 release.tag.version.major === 0),
                     });
                 }
+                else {
+                    logger_1.logger.info(`No release necessary for path: ${path}`);
+                }
             }
         }
         return releases;

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

@@ -102,6 +102,13 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
             throw new Error(`Could not find graph package for ${pkg.name}`);
         }
         const updatedPackage = pkg.clone();
+        // Update version of the package
+        const newVersion = updatedVersions.get(updatedPackage.name);
+        if (newVersion) {
+            logger_1.logger.info(`Updating ${updatedPackage.name} to ${newVersion}`);
+            updatedPackage.version = newVersion.toString();
+        }
+        // Update dependency versions
         for (const [depName, resolved] of graphPackage.localDependencies) {
             const depVersion = updatedVersions.get(depName);
             if (depVersion && resolved.type !== 'directory') {
@@ -116,7 +123,10 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
                     update.updater = new raw_content_1.RawContent(json_stringify_1.jsonStringify(updatedPackage.toJSON(), updatedPackage.rawContent));
                 }
                 else if (update.updater instanceof changelog_1.Changelog) {
-                    update.updater.changelogEntry = appendDependenciesSectionToChangelog(update.updater.changelogEntry, dependencyNotes);
+                    if (dependencyNotes) {
+                        update.updater.changelogEntry =
+                            appendDependenciesSectionToChangelog(update.updater.changelogEntry, dependencyNotes);
+                    }
                 }
                 return update;
             });
@@ -143,6 +153,12 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
             throw new Error(`Could not find graph package for ${pkg.name}`);
         }
         const updatedPackage = pkg.clone();
+        // Update version of the package
+        const newVersion = updatedVersions.get(updatedPackage.name);
+        if (newVersion) {
+            logger_1.logger.info(`Updating ${updatedPackage.name} to ${newVersion}`);
+            updatedPackage.version = newVersion.toString();
+        }
         for (const [depName, resolved] of graphPackage.localDependencies) {
             const depVersion = updatedVersions.get(depName);
             if (depVersion && resolved.type !== 'directory') {
@@ -173,7 +189,7 @@ class NodeWorkspace extends workspace_1.WorkspacePlugin {
                     createIfMissing: false,
                     updater: new changelog_1.Changelog({
                         version,
-                        changelogEntry: dependencyNotes,
+                        changelogEntry: appendDependenciesSectionToChangelog('', dependencyNotes),
                     }),
                 },
             ],

package/build/src/strategies/ocaml.js

@@ -61,6 +61,16 @@ class OCaml extends base_1.BaseStrategy {
                 }),
             });
         });
+        const opamLockedPaths = await this.github.findFilesByExtension('opam.locked', this.path);
+        opamLockedPaths.filter(notEsyLock).forEach(path => {
+            updates.push({
+                path: this.addPath(path),
+                createIfMissing: false,
+                updater: new opam_1.Opam({
+                    version,
+                }),
+            });
+        });
         updates.push({
             path: this.addPath('dune-project'),
             createIfMissing: false,

package/build/src/versioning-strategies/dependency-manifest.js

@@ -18,7 +18,7 @@ const version_1 = require("../version");
 const semver = require("semver");
 const default_1 = require("./default");
 const versioning_strategy_1 = require("../versioning-strategy");
-const DEPENDENCY_UPDATE_REGEX = /^deps: update dependency (.*) to (v.*)(\s\(#\d+\))?$/m;
+const DEPENDENCY_UPDATE_REGEX = /^deps: update dependency (.*) to (v[^\s]*)(\s\(#\d+\))?$/m;
 /**
  * This VersioningStrategy looks at `deps` type commits and tries to
  * mirror the semantic version bump for that dependency update. For

package/package.json

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