allow building a pull request if there are no conventional commits (#208)

Co-authored-by: meorphis <eric@stainless.com>

Author: meorphis

src/manifest.ts

@@ -835,14 +835,16 @@ export class Manifest {
         openPullRequests.find(pr => pr.headBranchName === branchName) ||
         snoozedPullRequests.find(pr => pr.headBranchName === branchName);
 
-      const releasePullRequest = await strategy.buildReleasePullRequest({
-        commits: pathCommits,
-        latestRelease,
-        draft: config.draftPullRequest ?? this.draftPullRequest,
-        labels: this.labels,
-        existingPullRequest: existingPR,
-        manifestPath: this.manifestPath,
-      });
+      const releasePullRequest = commitsPerPath[path]
+        ? await strategy.buildReleasePullRequest({
+            commits: pathCommits,
+            latestRelease,
+            draft: config.draftPullRequest ?? this.draftPullRequest,
+            labels: this.labels,
+            existingPullRequest: existingPR,
+            manifestPath: this.manifestPath,
+          })
+        : undefined;
       this.logger.debug(`path: ${path}`);
       this.logger.debug(
         `releasePullRequest.headRefName: ${releasePullRequest?.headRefName}`

src/strategies/base.ts

@@ -285,11 +285,7 @@ export abstract class BaseStrategy implements Strategy {
     manifestPath?: string;
   }): Promise<ReleasePullRequest | undefined> {
     const conventionalCommits = await this.postProcessCommits(commits);
-    this.logger.info(`Considering: ${commits.length} commits`);
-    if (commits.length === 0) {
-      this.logger.info(`No commits for path: ${this.path}, skipping`);
-      return undefined;
-    }
+    this.logger.info(`Considering: ${commits.length} conventional commits`);
 
     const component = await this.getComponent();
     this.logger.debug('component:', component);