fix: handle UNKNOWN status and lint fix

nodejs · Jan 15, 2025 · 5a6058f · 5a6058f
1 parent 12dcbf6
commit 5a6058f
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/lib/prepare_release.js b/lib/prepare_release.js
@@ -40,7 +40,7 @@ export default class ReleasePreparation extends Session {
     const { cli } = this;
 
     cli.warn(`PR#${pr.number} - ${pr.title} is not 'MERGEABLE'.
-      So, it will be skipped. Status: ${pr.mergeable}`);
+      Status: ${pr.mergeable}`);
   }
 
   async getOpenPRs(filterLabels) {
@@ -66,7 +66,6 @@ export default class ReleasePreparation extends Session {
     for (const pr of prs) {
       if (pr.mergeable !== 'MERGEABLE') {
         this.warnForNonMergeablePR(pr);
-        continue;
       }
       const cp = new CherryPick(pr.number, this.dir, cli, {
         owner: this.owner,
@@ -486,7 +485,7 @@ export default class ReleasePreparation extends Session {
     let releaseHeader = `## ${date}, Version ${newVersion}` +
                           ` ${releaseInfo}, @${username}\n`;
     if (isSecurityRelease) {
-      releaseHeader += '\nThis is a security release.';
+      releaseHeader += '\nThis is a security release.\n';
     }
 
     const topHeader =