Commit 9df9e9f8 authored by Gabriel Charette's avatar Gabriel Charette Committed by Commit Bot

Dump reviewers set when failing _check_reviewers

I've hit this assert a few times, seeing the output of the set usually
helps highlight quickly how my command-line arguments are broken.

Bug:
Change-Id: I8b80b635df91f2707a599a33ab9b4959b3c76614
Reviewed-on: https://chromium-review.googlesource.com/535834Reviewed-by: 's avatarAndrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
parent 44398fae
......@@ -185,7 +185,7 @@ class Database(object):
def _check_reviewers(self, reviewers):
_assert_is_collection(reviewers)
assert all(self.email_regexp.match(r) for r in reviewers)
assert all(self.email_regexp.match(r) for r in reviewers), reviewers
def _is_obj_covered_by(self, objname, reviewers):
reviewers = list(reviewers) + [EVERYONE]
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment