feat(raw-commits): allow hidding by subject pattern

[roggervalf]

Add a way to hide commits with specific pattern from the release notes. This is useful in order to avoid commits with specific messages that we don't want to include in release notes

[roggervalf]

please @dangreen when you get a chance 👀

[dangreen]

@roggervalf Hi. I think "types" is not right place for filtering by subject. We can add new option "ignore" to git-raw-commits to ignore commits by regex (here). And pass it option from preset

https://github.com/conventional-changelog/conventional-changelog/blob/master/packages/conventional-changelog-conventionalcommits/index.js#L15

{
  gitRawCommitsOpts: {
    ignore: config.ignoreCommits,
    noMerges: null
  }
}

WDYT?

[roggervalf]

@roggervalf Hi. I think "types" is not right place for filtering by subject. We can add new option "ignore" to git-raw-commits to ignore commits by regex (here). And pass it option from preset

https://github.com/conventional-changelog/conventional-changelog/blob/master/packages/conventional-changelog-conventionalcommits/index.js#L15

{
  gitRawCommitsOpts: {
    ignore: config.ignoreCommits,
    noMerges: null
  }
}

WDYT?

Thank you @dangreen, sounds good to me, I think I could address it

[dangreen]

Ignore commit by pattern is a rare case, so I don't want to add new dependency for it. Regex should fit you needs

[dangreen]

Let's use regex instead of glob

const ignoreRegex = typeof gitOpts.ignore === 'string'
  ? new RegExp(gitOpts.ignore)
  : gitOpts.ignore
const shouldNotIgnore = ignoreRegex
  ? chunk => !ignoreRegex.test(chunk.toString())
  : () => true
...
if (shouldNotIgnore(chunk)) {
  readable.push(chunk)
}

[roggervalf]

Thank you @dangreen, it sound good to me

[coveralls]

coverage: 89.471% (-0.05%) from 89.525% when pulling c55c5e4 on roggervalf:feat-allow-hidding-by-subject into 7ec8a18 on conventional-changelog:master.