Skip to content

Commit

Permalink
Remove redundant code in parse_commits
Browse files Browse the repository at this point in the history
There are two reasons for removing these lines of code

1. There are no push methods for strings (author_line/committer_line)
2. git does not support multiple authors/committers

Signed-off-by: Lance Chen <[email protected]>
  • Loading branch information
lancechentw committed Apr 6, 2014
1 parent d27c138 commit 9ab97cc
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions src/commit.coffee
Original file line number Diff line number Diff line change
Expand Up @@ -71,13 +71,9 @@ module.exports = class Commit
parents.push _.last lines.shift().split(" ")

author_line = lines.shift()
if !/^committer /.test(lines[0])
author_line.push lines.shift()
[author, authored_date] = @actor author_line

committer_line = lines.shift()
if lines[0] && !/(^encoding|^gpgsig)/.test(lines[0])
committer_line.push lines.shift()
[committer, committed_date] = @actor committer_line

gpgsig = []
Expand Down

0 comments on commit 9ab97cc

Please sign in to comment.