Skip to content
This repository has been archived by the owner on Nov 21, 2017. It is now read-only.

Commit

Permalink
refactor(naming): releaseAs => releaseType
Browse files Browse the repository at this point in the history
BREAKING CHANGE: `releaseAs` => `releaseType`

Fixes #15
  • Loading branch information
stevemao committed Aug 13, 2016
1 parent 3ec3d7d commit 1476f1e
Show file tree
Hide file tree
Showing 5 changed files with 30 additions and 30 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ var conventionalRecommendedBump = require('conventional-recommended-bump');
conventionalRecommendedBump({
preset: 'angular'
}, function(err, result) {
console.log(result.releaseAs);
console.log(result.releaseType);
//=> 'major'
});
```
Expand Down Expand Up @@ -91,7 +91,7 @@ Type: `function`

Object includes what's returned by `whatBump` and

####### releaseAs
####### releaseType

Type: `string` Possible values: `'major'`, `'minor'` and `'patch'`

Expand Down
4 changes: 2 additions & 2 deletions cli.js
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,8 @@ conventionalRecommendedBump(options, flags, function(err, data) {
process.exit(1);
}

if (data.releaseAs) {
console.log(data.releaseAs);
if (data.releaseType) {
console.log(data.releaseType);
}

if (flags.verbose && data.reason) {
Expand Down
2 changes: 1 addition & 1 deletion index.js
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ function conventionalRecommendedBump(options, parserOpts, cb) {
}

if (result && result.level != null) {
result.releaseAs = VERSIONS[result.level];
result.releaseType = VERSIONS[result.level];
} else if (result == null) {
result = {};
}
Expand Down
36 changes: 18 additions & 18 deletions test/angular.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,11 @@ describe('preset', function() {
});

it('should release as minor', function(done) {
conventionalRecommendedBump(opts, function(err, releaseAs) {
equal(releaseAs, {
conventionalRecommendedBump(opts, function(err, releaseType) {
equal(releaseType, {
level: 1,
reason: 'There are 0 BREAKING CHANGES and 1 features',
releaseAs: 'minor'
releaseType: 'minor'
});

done();
Expand All @@ -41,11 +41,11 @@ describe('preset', function() {
it('should merge parserOpts', function(done) {
conventionalRecommendedBump(opts, {
headerPattern: /^(\w*)\: (.*)$/,
}, function(err, releaseAs) {
equal(releaseAs, {
}, function(err, releaseType) {
equal(releaseType, {
level: 2,
reason: 'There are 0 BREAKING CHANGES and 0 features',
releaseAs: 'patch'
releaseType: 'patch'
});

done();
Expand All @@ -56,11 +56,11 @@ describe('preset', function() {
writeFileSync('test4', '');
// fix this until https://github.com/arturadib/shelljs/issues/175 is solved
child.exec('git add --all && git commit -m "feat(): amazing new module" -m "BREAKING CHANGE: Not backward compatible."', function() {
conventionalRecommendedBump(opts, function(err, releaseAs) {
equal(releaseAs, {
conventionalRecommendedBump(opts, function(err, releaseType) {
equal(releaseType, {
level: 0,
reason: 'There are 1 BREAKING CHANGES and 1 features',
releaseAs: 'major'
releaseType: 'major'
});

done();
Expand All @@ -72,11 +72,11 @@ describe('preset', function() {
writeFileSync('test5', '');
// fix this until https://github.com/arturadib/shelljs/issues/175 is solved
child.exec('git add --all && git commit -m "feat(): another amazing new module" -m "Super backward compatible."', function() {
conventionalRecommendedBump(opts, function(err, releaseAs) {
equal(releaseAs, {
conventionalRecommendedBump(opts, function(err, releaseType) {
equal(releaseType, {
level: 0,
reason: 'There are 1 BREAKING CHANGES and 2 features',
releaseAs: 'major'
releaseType: 'major'
});

done();
Expand All @@ -89,11 +89,11 @@ describe('preset', function() {
child.exec('git rev-parse HEAD~1', function(err, hash) {
// fix this until https://github.com/arturadib/shelljs/issues/175 is solved
child.exec('git add --all && git commit -m "revert: feat(): amazing new module" -m "This reverts commit ' + hash.trim() + '."', function() {
conventionalRecommendedBump(opts, function(err, releaseAs) {
equal(releaseAs, {
conventionalRecommendedBump(opts, function(err, releaseType) {
equal(releaseType, {
level: 1,
reason: 'There are 0 BREAKING CHANGES and 2 features',
releaseAs: 'minor'
releaseType: 'minor'
});

done();
Expand All @@ -106,11 +106,11 @@ describe('preset', function() {
conventionalRecommendedBump({
preset: 'angular',
ignoreReverted: false
}, function(err, releaseAs) {
equal(releaseAs, {
}, function(err, releaseType) {
equal(releaseType, {
level: 0,
reason: 'There are 1 BREAKING CHANGES and 2 features',
releaseAs: 'major'
releaseType: 'major'
});

done();
Expand Down
14 changes: 7 additions & 7 deletions test/test.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ describe('conventional-recommended-bump', function() {
it('should return `{}` if no `whatBump` is found', function(done) {
shell.exec('git add --all && git commit -m"First commit"');

conventionalRecommendedBump({}, function(err, releaseAs) {
equal(releaseAs, {});
conventionalRecommendedBump({}, function(err, releaseType) {
equal(releaseType, {});

done();
});
Expand All @@ -44,8 +44,8 @@ describe('conventional-recommended-bump', function() {
test: 'test'
};
}
}, function(err, releaseAs) {
equal(releaseAs, {
}, function(err, releaseType) {
equal(releaseType, {
test: 'test'
});

Expand All @@ -58,10 +58,10 @@ describe('conventional-recommended-bump', function() {
whatBump: function() {
return 0;
}
}, function(err, releaseAs) {
equal(releaseAs, {
}, function(err, releaseType) {
equal(releaseType, {
level: 0,
releaseAs: 'major'
releaseType: 'major'
});

done();
Expand Down

0 comments on commit 1476f1e

Please sign in to comment.