Fixes #4: Add branch name in push message, tidy up push hook;
This commit is contained in:
parent
9772ad859f
commit
0f103fabfe
23
index.js
23
index.js
@ -28,15 +28,28 @@ var handlers = {
|
|||||||
push: function(body) {
|
push: function(body) {
|
||||||
var user = body.user_name;
|
var user = body.user_name;
|
||||||
var projectName = body.project.name;
|
var projectName = body.project.name;
|
||||||
|
|
||||||
var commits = body.commits;
|
var commits = body.commits;
|
||||||
|
var numCommits = body.total_commits_count;
|
||||||
|
|
||||||
var msg = ['push on ' + projectName + ': ' + user + ' pushed ' + commits.length + ' commits.'];
|
var branchName = body.ref.replace('refs/heads/', '');
|
||||||
|
|
||||||
if (commits.length < 4) {
|
var msg = [];
|
||||||
msg = msg.concat(commits.map(formatCommit));
|
if (!numCommits) {
|
||||||
|
// Special case: a branch was created or deleted.
|
||||||
|
var action = 'created';
|
||||||
|
if (body.after === '0000000000000000000000000000000000000000')
|
||||||
|
action = 'deleted';
|
||||||
|
msg.push(projectName + ': ' + user + ' ' + action + ' branch ' + branchName);
|
||||||
} else {
|
} else {
|
||||||
msg.push(formatCommit(commits[0]) + ' ...');
|
var maybeS = numCommits === 1 ? '' : 's';
|
||||||
msg.push(formatCommit(commits[commits.length - 1]));
|
msg.push('push on ' + projectName + '@' + branchName + ': ' + user + ' pushed ' + commits.length + ' commit' + maybeS + '.');
|
||||||
|
if (numCommits < 4) {
|
||||||
|
msg = msg.concat(commits.map(formatCommit));
|
||||||
|
} else {
|
||||||
|
msg.push(formatCommit(commits[0]) + ' ...');
|
||||||
|
msg.push(formatCommit(commits[commits.length - 1]));
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
return msg;
|
return msg;
|
||||||
|
Loading…
Reference in New Issue
Block a user