Add mechnism to ignore commits in release notes (#1330)

* Add mechnism to ignore commits in release notes. Ignore 'e5b0a312ddb1d508b2d14b4c101b894f2a4c7356' because it was committed without a PR. * Use full hash, filter out at the right spot
chanzuckerberg · Jan 8, 2025 · 5fd6464 · 5fd6464
1 parent b3072d5
commit 5fd6464
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/.github/release-notes.config.js b/.github/release-notes.config.js
@@ -40,4 +40,11 @@ module.exports = {
       labels: [],
     },
   ],
+
+  /**
+   * Ignore specific commits that cause problems with the release notes workflow.
+   */
+  ignoreCommits: [
+    "e5b0a312ddb1d508b2d14b4c101b894f2a4c7356", // committed without PR
+  ],
 }
diff --git a/.github/release-notes.js b/.github/release-notes.js
@@ -18,7 +18,7 @@ async function getCommitsSinceLastRelease(exec) {
 
   await exec.exec(
     'git',
-    ['log', '--pretty=format:"%h"', `${LATEST_RELEASE}..${TARGET_BRANCH}`],
+    ['log', '--pretty=format:"%H"', `${LATEST_RELEASE}..${TARGET_BRANCH}`],
     {
       listeners: {
         stdout(data) {
@@ -36,6 +36,7 @@ async function getCommitsSinceLastRelease(exec) {
     .flatMap(hash => hash.split('\n'))
     .filter(Boolean)
     .map(hash => hash.replaceAll('"', ''))
+    .filter(hash => !config.ignoreCommits.includes(hash))
 }
 
 /**