Show More
@@ -154,9 +154,10 b' def get_diff_info(' | |||||
154 |
|
154 | |||
155 | commits = [] |
|
155 | commits = [] | |
156 | if get_commit_authors: |
|
156 | if get_commit_authors: | |
|
157 | log.debug('Obtaining commit authors from set of commits') | |||
157 | commits = target_scm.compare( |
|
158 | commits = target_scm.compare( | |
158 | target_ref, source_ref, source_scm, merge=True, |
|
159 | target_ref, source_ref, source_scm, merge=True, | |
159 | pre_load=["author"]) |
|
160 | pre_load=["author", "date", "message", "branch", "parents"]) | |
160 |
|
161 | |||
161 | for commit in commits: |
|
162 | for commit in commits: | |
162 | user = User.get_from_cs_author(commit.author) |
|
163 | user = User.get_from_cs_author(commit.author) | |
@@ -165,6 +166,7 b' def get_diff_info(' | |||||
165 |
|
166 | |||
166 | # lines |
|
167 | # lines | |
167 | if get_authors: |
|
168 | if get_authors: | |
|
169 | log.debug('Calculating authors of changed files') | |||
168 | target_commit = source_repo.get_commit(ancestor_id) |
|
170 | target_commit = source_repo.get_commit(ancestor_id) | |
169 |
|
171 | |||
170 | for fname, lines in changed_lines.items(): |
|
172 | for fname, lines in changed_lines.items(): | |
@@ -190,6 +192,8 b' def get_diff_info(' | |||||
190 | author_counts[author] = author_counts.get(author, 0) + 1 |
|
192 | author_counts[author] = author_counts.get(author, 0) + 1 | |
191 | email_counts[email] = email_counts.get(email, 0) + 1 |
|
193 | email_counts[email] = email_counts.get(email, 0) + 1 | |
192 |
|
194 | |||
|
195 | log.debug('Default reviewers processing finished') | |||
|
196 | ||||
193 | return { |
|
197 | return { | |
194 | 'commits': commits, |
|
198 | 'commits': commits, | |
195 | 'files': all_files_changes, |
|
199 | 'files': all_files_changes, |
General Comments 0
You need to be logged in to leave comments.
Login now