Merge pull request #25 from btnguyen2k/dev

Fix: duplicated commit messages
btnguyen2k · Aug 11, 2023 · f462bdd · f462bdd
2 parents 4f2317c + 51efbc7
commit f462bdd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/RELEASE-NOTES.md b/RELEASE-NOTES.md
@@ -4,7 +4,6 @@
 
 ### Fixed/Improvement
 
-- Improvement: prevent duplicated commit messages
 - Improvement: prevent duplicated commit messages
 
 ## 2023-08-11 - v2.0.0

diff --git a/src/index.js b/src/index.js
@@ -153,11 +153,9 @@ async function computeReleaseNotes(octokit) {
     const commits = await utils.getAllCommits(octokit, {...filterCommits, sha: branch})
     for (const commit of commits) {
       const commitMsg = commit.commit.message.trim()
-      for (const msg of messages) {
-        if (msg === commitMsg) {
-          // prevent duplicated messages
-          continue
-        }
+      if (messages.some(msg => msg === commitMsg)) {
+        // prevent duplicated messages
+        continue
       }
       messages.push(commitMsg)
     }