diff options
author | mtklein <mtklein@chromium.org> | 2015-07-28 08:54:12 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-07-28 08:54:12 -0700 |
commit | bda12677704bc1c19b156236f4f8b08d2fb0b4fc (patch) | |
tree | cab8d17ae3e0ba125265f83ed7b205a0a8b4867f /PRESUBMIT.py | |
parent | 562fe4767cc73e08a4e039362bc0336aea66ecfb (diff) |
update presubmit to know about include/private
BUG=skia:4126
Review URL: https://codereview.chromium.org/1259953004
Diffstat (limited to 'PRESUBMIT.py')
-rw-r--r-- | PRESUBMIT.py | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/PRESUBMIT.py b/PRESUBMIT.py index 5599c316c3..5d0d6bbe8c 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -279,8 +279,10 @@ def _CheckLGTMsForPublicAPI(input_api, output_api): affected_file_path = affected_file.LocalPath() file_path, file_ext = os.path.splitext(affected_file_path) # We only care about files that end in .h and are under the top-level - # include dir. - if file_ext == '.h' and 'include' == file_path.split(os.path.sep)[0]: + # include dir, but not include/private. + if (file_ext == '.h' and + 'include' == file_path.split(os.path.sep)[0] and + 'private' not in file_path): requires_owner_check = True if not requires_owner_check: @@ -328,8 +330,12 @@ def _CheckLGTMsForPublicAPI(input_api, output_api): if not lgtm_from_owner: results.append( output_api.PresubmitError( - 'Since the CL is editing public API, you must have an LGTM from ' - 'one of: %s' % str(PUBLIC_API_OWNERS))) + "If this CL adds to or changes Skia's public API, you need an LGTM " + "from any of %s. If this CL only removes from or doesn't change " + "Skia's public API, please add a short note to the CL saying so " + "and add one of those reviewers on a TBR= line. If you don't know " + "if this CL affects Skia's public API, treat it like it does." + % str(PUBLIC_API_OWNERS))) return results |