From 9df9e9f8f9b218862b41817dd7604251484cc884 Mon Sep 17 00:00:00 2001 From: Gabriel Charette Date: Wed, 14 Jun 2017 15:44:50 -0400 Subject: [PATCH] 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/535834 Reviewed-by: Andrii Shyshkalov Commit-Queue: Gabriel Charette --- owners.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/owners.py b/owners.py index fd0c800191..801812710a 100644 --- a/owners.py +++ b/owners.py @@ -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]