Merge 5f05c08 into 169fbcf

presets/eslint.js

@@ -0,0 +1,36 @@
+var presetOpts = {
+  whatBump: function(commits) {
+    var level = 2;
+    var breakings = 0;
+    var features = 0;
+
+    commits.forEach(function(commit) {
+      commit.tag = (commit.tag || '').toLowerCase();
+      if (commit.tag === 'breaking') {
+        breakings += 1;
+        level = 0;
+      } else if (commit.tag === 'feat') {
+        features += 1;
+        if (level === 2) {
+          level = 1;
+        }
+      }
+    });
+
+    return {
+      level: level,
+      reason: 'There are ' + breakings + ' BREAKING CHANGES and ' + features + ' features'
+    };
+  },
+  parserOpts: {
+    headerPattern: /^(\w*)\: (.*?)(?:\((.*)\))?$/,
+    headerCorrespondence: [
+      'tag',
+      'message'
+    ],
+    revertPattern: /^revert:\s([\s\S]*?)\s*This reverts commit (\w*)\./,
+    revertCorrespondence: ['header', 'hash']
+  }
+};
+
+module.exports = presetOpts;

test/eslint.js

@@ -0,0 +1,155 @@
+'use strict';
+var execSync = require('child_process').execSync;
+var conventionalRecommendedBump = require('../');
+var equal = require('core-assert').deepStrictEqual;
+var shell = require('shelljs');
+var writeFileSync = require('fs').writeFileSync;
+var betterThanBefore = require('better-than-before')();
+var preparing = betterThanBefore.preparing;
+
+betterThanBefore.setups([
+  function() { // 1
+    shell.mkdir('eslint');
+    shell.cd('eslint');
+    shell.exec('git init');
+    writeFileSync('test1', '');
+    shell.exec('git add --all && git commit -m "chore: first commit"');
+    writeFileSync('test2', '');
+    shell.exec('git add --all && git commit -m "feat: new feature"');
+    writeFileSync('test3', '');
+    shell.exec('git add --all && git commit -m "update: make it faster"');
+  },
+  function() { // 2
+    writeFileSync('test4', '');
+    execSync('git add --all && git commit -m "breaking: amazing new module" -m "This breaks everything!"');
+  },
+  function() { // 3
+    writeFileSync('test5', '');
+    execSync('git add --all && git commit -m "feat: another amazing new module" -m "Super backward compatible."');
+  },
+  function() { // 4
+    writeFileSync('test6', '');
+    var hash = execSync('git rev-parse HEAD~1').toString();
+    execSync('git add --all && git commit -m "revert: breaking: amazing new module" -m "This reverts commit ' + hash.trim() + '."');
+  },
+  function() { // 5
+    writeFileSync('test7', '');
+    execSync('git add --all && git commit -m "feat: make BREAKING CHANGE more forgiving\n\nPeople might type BREAKING CHANGES unintentionally. EG: https://github.com/eslint/eslint/commit/098b461"');
+  }
+]);
+
+betterThanBefore.tearsWithJoy(function() {
+  shell.cd('../');
+  shell.rm('-rf', 'eslint');
+});
+
+describe('preset', function() {
+  describe('eslint', function() {
+    var opts = {
+      preset: 'eslint'
+    };
+
+    it('should release as minor', function(done) {
+      preparing(1);
+
+      conventionalRecommendedBump(opts, function(err, releaseType) {
+        equal(releaseType, {
+          level: 1,
+          reason: 'There are 0 BREAKING CHANGES and 1 features',
+          releaseType: 'minor'
+        });
+
+        done();
+      });
+    });
+
+    it('should merge parserOpts', function(done) {
+      preparing(1);
+
+      conventionalRecommendedBump(opts, {
+        headerPattern: /^(\w*)\(.*\)\: (.*)$/,
+      }, function(err, releaseType) {
+        equal(releaseType, {
+          level: 2,
+          reason: 'There are 0 BREAKING CHANGES and 0 features',
+          releaseType: 'patch'
+        });
+
+        done();
+      });
+    });
+
+    it('should release as major', function(done) {
+      preparing(2);
+
+      conventionalRecommendedBump(opts, function(err, releaseType) {
+        equal(releaseType, {
+          level: 0,
+          reason: 'There are 1 BREAKING CHANGES and 1 features',
+          releaseType: 'major'
+        });
+
+        done();
+      });
+    });
+
+    it('should release as major even after a feature', function(done) {
+      preparing(3);
+
+      conventionalRecommendedBump(opts, function(err, releaseType) {
+        equal(releaseType, {
+          level: 0,
+          reason: 'There are 1 BREAKING CHANGES and 2 features',
+          releaseType: 'major'
+        });
+
+        done();
+      });
+    });
+
+    it('should ignore a reverted commit', function(done) {
+      preparing(4);
+
+      conventionalRecommendedBump(opts, function(err, releaseType) {
+        equal(releaseType, {
+          level: 1,
+          reason: 'There are 0 BREAKING CHANGES and 2 features',
+          releaseType: 'minor'
+        });
+
+        done();
+      });
+    });
+
+    it('should not ignore a reverted commit', function(done) {
+      preparing(4);
+
+      conventionalRecommendedBump({
+        preset: 'eslint',
+        ignoreReverted: false
+      }, function(err, releaseType) {
+        equal(releaseType, {
+          level: 0,
+          reason: 'There are 1 BREAKING CHANGES and 2 features',
+          releaseType: 'major'
+        });
+
+        done();
+      });
+    });
+
+    it('should not be major', function(done) {
+      preparing(5);
+
+      conventionalRecommendedBump(opts, function(err, releaseType) {
+        equal(releaseType, {
+          level: 1,
+          reason: 'There are 0 BREAKING CHANGES and 3 features',
+          releaseType: 'minor'
+        });
+
+        done();
+      });
+    });
+  });
+});