refactor: use object destructuring rather than omit

Author: pvdlg

lib/analyze-commit.js

@@ -1,4 +1,4 @@
-const {isMatchWith, isRegExp, omit} = require('lodash');
+const {isMatchWith, isRegExp} = require('lodash');
 const debug = require('debug')('semantic-release:commit-analyzer');
 const RELEASE_TYPES = require('./default-release-types');
 const compareReleaseTypes = require('./compare-release-types');
@@ -15,11 +15,13 @@ module.exports = (releaseRules, commit) => {
 
   releaseRules
     .filter(
-      rule =>
-        (!rule.breaking || (commit.notes && commit.notes.length > 0)) &&
+      ({breaking, release, ...rule}) =>
+        // If the rule is not `breaking` or the commit doesn't have a breaking change note
+        (!breaking || (commit.notes && commit.notes.length > 0)) &&
+        // Otherwise match the regular rules
         isMatchWith(
           commit,
-          omit(rule, ['release', 'breaking']),
+          rule,
           (obj, src) =>
             /^\/.*\/$/.test(src) || isRegExp(src) ? new RegExp(/^\/(.*)\/$/.exec(src)[1]).test(obj) : undefined
         )