Commit 5861efbd authored by ilevy@chromium.org's avatar ilevy@chromium.org

Silence Valid authors debug msg in presubmit

Silence the valid authors debug message, unless the check fails.
The message takes a lot of space because the author file is big, and
is not especially helpful (just lists parse results from the authors
file).

NOTRY=True


Review URL: https://chromiumcodereview.appspot.com/11782010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175360 0039d316-1c4b-4281-b951-d872f2087c98
parent f1cabab0
......@@ -774,9 +774,8 @@ def _CheckAuthorizedAuthor(input_api, output_api):
input_api.re.match(r'[^#]+\s+\<(.+?)\>\s*$', line)
for line in open(authors_path))
valid_authors = [item.group(1).lower() for item in valid_authors if item]
if input_api.verbose:
print 'Valid authors are %s' % ', '.join(valid_authors)
if not any(fnmatch.fnmatch(author.lower(), valid) for valid in valid_authors):
input_api.logging.info('Valid authors are %s', ', '.join(valid_authors))
return [output_api.PresubmitPromptWarning(
('%s is not in AUTHORS file. If you are a new contributor, please visit'
'\n'
......
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