Index: PRESUBMIT.py |
diff --git a/PRESUBMIT.py b/PRESUBMIT.py |
index 6fface8db19a582e7b7d8188ec5be9aed076d5a0..b11ed8d40d31f70cfa535ce9e0e80d3ea7c3599d 100644 |
--- a/PRESUBMIT.py |
+++ b/PRESUBMIT.py |
@@ -111,9 +111,7 @@ def _CheckAuthorizedAuthor(input_api, output_api): |
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( |
- True for valid in valid_authors |
- if fnmatch.fnmatch(author.lower(), valid)): |
+ if not any(fnmatch.fnmatch(author.lower(), valid) for valid in valid_authors): |
Dirk Pranke
2011/06/14 17:44:27
why is this fnmatch? authors aren't filenames ...
M-A Ruel
2011/06/14 18:06:30
Yes but using wildcards is *much* easier than usin
|
return [output_api.PresubmitPromptWarning( |
('%s is not in AUTHORS file. If you are a new contributor, please visit' |
'\n' |