Skip to content

Commit

Permalink
feat(changelog): Allow 'enforce' option: Adds a git hook for commit c…
Browse files Browse the repository at this point in the history
…onventions
  • Loading branch information
0x6a68 authored and btford committed May 30, 2013
1 parent 4a06569 commit 1cbe92c
Show file tree
Hide file tree
Showing 3 changed files with 120 additions and 1 deletion.
3 changes: 2 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,8 @@ A string which contains the value of the version which is used by grunt-conventi
if now version is specified, grunt-conventional-changelog expects a pkg object which has a version property and uses that.
>>>>>>> feat(changelog): add optional 'version', incase not package.json

### enforce
Enforce commit convention by using a commit-hook. based on [this](https://github.com/angular/angular.js/blob/master/validate-commit-msg.js). By Default `false`.

## Usage Examples

Expand Down
13 changes: 13 additions & 0 deletions tasks/changelog.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ module.exports = function (grunt) {
var options = this.options({
// dest: 'CHANGELOG.md',
prepend: true, // false to append
enforce: false
});

var githubRepo;
Expand All @@ -30,6 +31,18 @@ module.exports = function (grunt) {
} else {
githubRepo = options.github;
}

// ensure commit convention, by using a commit hook
var gitHookFile = '.git/hooks/commit-msg';
if(options.enforce) {
// if no commit hook is found, copy from template
if(!grunt.file.exists(gitHookFile)) {
grunt.file.copy(__dirname + '/../validate-commit-msg.js', gitHookFile);
// need to ensure the hook is executable
fs.chmodSync(gitHookFile, '0755');
}
}

function fixGithubRepo(githubRepo) {
//User could set option eg 'github: "btford/grunt-conventional-changelog'
if (githubRepo.indexOf('github.com') === -1) {
Expand Down
105 changes: 105 additions & 0 deletions validate-commit-msg.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,105 @@
#!/usr/bin/env node

/**
* Git COMMIT-MSG hook for validating commit message
* See https://docs.google.com/document/d/1rk04jEuGfk9kYzfqCuOlPTSJw3hEDZJTBN5E5f1SALo/edit
*
* Installation:
* >> cd <angular-repo>
* >> ln -s ../../validate-commit-msg.js .git/hooks/commit-msg
*/
var fs = require('fs');
var util = require('util');


var MAX_LENGTH = 70;
var PATTERN = /^(?:fixup!\s*)?(\w*)(\(([\w\$\.\-\*/]*)\))?\: (.*)$/;
var IGNORED = /^WIP\:/;
var TYPES = {
feat: true,
fix: true,
docs: true,
style: true,
refactor: true,
test: true,
chore: true,
revert: true
};


var error = function() {
// gitx does not display it
// http://gitx.lighthouseapp.com/projects/17830/tickets/294-feature-display-hook-error-message-when-hook-fails
// https://groups.google.com/group/gitx/browse_thread/thread/a03bcab60844b812
console.error('INVALID COMMIT MSG: ' + util.format.apply(null, arguments));
};


var validateMessage = function(message) {
var isValid = true;

if (IGNORED.test(message)) {
console.log('Commit message validation ignored.');
return true;
}

if (message.length > MAX_LENGTH) {
error('is longer than %d characters !', MAX_LENGTH);
isValid = false;
}

var match = PATTERN.exec(message);

if (!match) {
error('does not match "<type>(<scope>): <subject>" ! was: ' + message);
return false;
}

var type = match[1];
var scope = match[3];
var subject = match[4];

if (!TYPES.hasOwnProperty(type)) {
error('"%s" is not allowed type !', type);
return false;
}

// Some more ideas, do want anything like this ?
// - allow only specific scopes (eg. fix(docs) should not be allowed ?
// - auto correct the type to lower case ?
// - auto correct first letter of the subject to lower case ?
// - auto add empty line after subject ?
// - auto remove empty () ?
// - auto correct typos in type ?
// - store incorrect messages, so that we can learn

return isValid;
};


var firstLineFromBuffer = function(buffer) {
return buffer.toString().split('\n').shift();
};



// publish for testing
exports.validateMessage = validateMessage;

// hacky start if not run by jasmine :-D
if (process.argv.join('').indexOf('jasmine-node') === -1) {
var commitMsgFile = process.argv[2];
var incorrectLogFile = commitMsgFile.replace('COMMIT_EDITMSG', 'logs/incorrect-commit-msgs');

fs.readFile(commitMsgFile, function(err, buffer) {
var msg = firstLineFromBuffer(buffer);

if (!validateMessage(msg)) {
fs.appendFile(incorrectLogFile, msg + '\n', function() {
process.exit(1);
});
} else {
process.exit(0);
}
});
}

0 comments on commit 1cbe92c

Please sign in to comment.