default-reviewers: fixed problems with new diff format for more advanced default reviewer rules.
marcink -
r4385:cf2c34da stable
Not Reviewed
Show More
Add another comment
TODOs: 0 unresolved 0 Resolved
COMMENTS: 0 General 0 Inline
@@ -4962,7 +4962,9 class RepoReviewRule(Base, BaseModel):
4962 else:
4962 else:
4963 file_pattern = glob2re(self.file_pattern)
4963 file_pattern = glob2re(self.file_pattern)
4964 file_regex = re.compile(file_pattern)
4964 file_regex = re.compile(file_pattern)
4965 for filename in files_changed:
4965 for file_data in files_changed:
4966 filename = file_data.get('filename')
4967
4966 if file_regex.search(filename):
4968 if file_regex.search(filename):
4967 files_matches = True
4969 files_matches = True
4968 break
4970 break
@@ -186,7 +186,7 def get_diff_info(
186 email = commit.author_email
186 email = commit.author_email
187 user = User.get_from_cs_author(author)
187 user = User.get_from_cs_author(author)
188 if user:
188 if user:
189 user_counts[user] = user_counts.get(user, 0) + 1
189 user_counts[user.user_id] = user_counts.get(user.user_id, 0) + 1
190 author_counts[author] = author_counts.get(author, 0) + 1
190 author_counts[author] = author_counts.get(author, 0) + 1
191 email_counts[email] = email_counts.get(email, 0) + 1
191 email_counts[email] = email_counts.get(email, 0) + 1
192
192
Comments 0
You need to be logged in to leave comments. Login now