Apply ignoreCommitPattern earlier in the parsing logic

Fixes #195
Sort of fixes #187 too

src/commits.js

@@ -35,6 +35,7 @@ const parseCommits = (string, options = {}) => {
     .split(COMMIT_SEPARATOR)
     .slice(1)
     .map(commit => parseCommit(commit, options))
+    .filter(commit => filterCommit(commit, options))
 }
 
 const parseCommit = (commit, options = {}) => {
@@ -133,6 +134,13 @@ const getMerge = (commit, message, options = {}) => {
   return null
 }
 
+const filterCommit = (commit, { ignoreCommitPattern }) => {
+  if (ignoreCommitPattern && new RegExp(ignoreCommitPattern).test(commit.subject)) {
+    return false
+  }
+  return true
+}
+
 module.exports = {
   COMMIT_SEPARATOR,
   MESSAGE_SEPARATOR,

src/releases.js

@@ -13,7 +13,7 @@ const parseReleases = async (tags, options, onParsed) => {
     const { message } = commits[0] || { message: null }
     const breakingCount = commits.filter(c => c.breaking).length
     const filteredCommits = commits
-      .filter(filterCommits(options, merges))
+      .filter(filterCommits(merges))
       .sort(sortCommits(options))
       .slice(0, getCommitLimit(options, emptyRelease, breakingCount))
 
@@ -29,17 +29,14 @@ const parseReleases = async (tags, options, onParsed) => {
   }))
 }
 
-const filterCommits = ({ ignoreCommitPattern }, merges) => commit => {
+const filterCommits = merges => commit => {
   if (commit.fixes || commit.merge) {
     // Filter out commits that already appear in fix or merge lists
     return false
   }
   if (commit.breaking) {
     return true
   }
-  if (ignoreCommitPattern && new RegExp(ignoreCommitPattern).test(commit.subject)) {
-    return false
-  }
   if (semver.valid(commit.subject)) {
     // Filter out version commits
     return false