[Pkg-javascript-commits] [node-coveralls] 200/332: Fix bug #86

[Bastien Roucariès]

This is an automated email from the git hooks/post-receive script.

rouca pushed a commit to branch master
in repository node-coveralls.

commit ebb47a974bc0665419cdf9d5c0ed4c4ed95636c8
Author: Hauke Henningsen <sqrt at entless.org>
Date:   Mon Apr 20 23:17:40 2015 +0200

    Fix bug #86
    
    Instead of matching the git commit file for emails which contain
    at least one character, match for “emails” which also may be empty.
    
    Also, use greedy regex matching in those places which is likely
    to be more performant than non-greedy matching.
---
 lib/fetchGitData.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/fetchGitData.js b/lib/fetchGitData.js
index 67b6190..a6abba4 100644
--- a/lib/fetchGitData.js
+++ b/lib/fetchGitData.js
@@ -59,7 +59,7 @@ function fetchBranch(git, cb) {
   });
 }
 
-var REGEX_COMMIT_DETAILS = /\nauthor (.+?) <(.+?)>.+\ncommitter (.+?) <(.+?)>.+\n?[\S\s]+?\n\n(.*)/m;
+var REGEX_COMMIT_DETAILS = /\nauthor (.+?) <([^>]*)>.+\ncommitter (.+?) <([^>]*)>.+\n?[\S\s]+?\n\n(.*)/m;
 
 function fetchHeadDetails(git, cb) {
   exec('git cat-file -p ' + git.head.id, function(err, response) {

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/node-coveralls.git