Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(474)

Unified Diff: pending_manager.py

Issue 6015008: Make many operations quieter. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/commit-queue
Patch Set: Rebase against trunk Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « commit_queue.py ('k') | rietveld.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pending_manager.py
diff --git a/pending_manager.py b/pending_manager.py
index 15574dcf4f14aa4fe4b6858d3386ba939216e40e..22df74bbb9c1f3a788d58d221d43b8b729f8eef6 100644
--- a/pending_manager.py
+++ b/pending_manager.py
@@ -202,10 +202,12 @@ class PendingManager(object):
"""Returns True if this pending is worth looking at."""
if not any(re.match(r, pending.base_url) for r in self.project_bases):
# Not a repository managed by this instance, silently ignore.
+ logging.info('%s not in whitelist' % pending.base_url)
return False
if any(re.match(r, pending.owner) for r in self.author_white_list):
# Can't commit because the owner is not on the whitelist.
+ logging.info('%s not in whitelist' % pending.owner)
return False
# Need at least one reviewer matching at least one regexp in
« no previous file with comments | « commit_queue.py ('k') | rietveld.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698