diff options
author | Eric Boren <borenet@google.com> | 2018-01-02 13:29:21 -0500 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-01-02 18:47:01 +0000 |
commit | dd98829c74120ac2e2bea79c02482a4f7c37d248 (patch) | |
tree | 96435700195a73178db845a4ef5461f3d82ce3c5 /PRESUBMIT.py | |
parent | 2a8275b7825e61c3bed5b1e86c3d70408d9ccf3a (diff) |
Skip AUTHORS check for Skia service accounts
Bug: skia:
Change-Id: Id9f3b6d0bc22993f5cfc9019354c9299441db05f
Reviewed-on: https://skia-review.googlesource.com/90261
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Eric Boren <borenet@google.com>
Diffstat (limited to 'PRESUBMIT.py')
-rw-r--r-- | PRESUBMIT.py | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/PRESUBMIT.py b/PRESUBMIT.py index a5cc808efe..487da42ac3 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -60,6 +60,8 @@ PATH_PREFIX_TO_EXTRA_TRYBOTS = { # 'src/image/SkImage_Base.h': 'master5:pqr,stu;master1:abc1;master2:def', } +SERVICE_ACCOUNT_SUFFIX = '@skia-buildbots.google.com.iam.gserviceaccount.com' + def _CheckChangeHasEol(input_api, output_api, source_file_filter=None): """Checks that files end with atleast one \n (LF).""" @@ -336,6 +338,11 @@ def _CheckOwnerIsInAuthorsFile(input_api, output_api): cr = CodeReview(input_api) owner_email = cr.GetOwnerEmail() + + # Service accounts don't need to be in AUTHORS. + if owner_email.endswith(SERVICE_ACCOUNT_SUFFIX): + return results + try: authors_content = '' for line in open(AUTHORS_FILE_NAME): |