diff options
| author |  Mike Klein <mtklein@google.com> | 2015-07-01 16:35:59 -0400 |
|---|---|---|
| committer | Mike Klein <mtklein@google.com> | 2015-07-01 16:35:59 -0400 |
| commit | 478c9e4851eff5feb74dd2f8a14143bec569e1a8 (patch) | |
| tree | 9acb07514c74296fe7b2b9dd17fa8a6ed907cf30 /PRESUBMIT.py | |
| parent | aa2a7de0ffaf8f864a7afc68bbd34ccb25876246 (diff) | |
Revert "Move headers used by headers in include/ to include/private."
This reverts commit 928e16565f3e69cd6661e9ad300ac17e3f33c0c8.
BUG=skia:
Review URL: https://codereview.chromium.org/1213093004.
Diffstat (limited to 'PRESUBMIT.py')
| -rw-r--r-- | PRESUBMIT.py | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/PRESUBMIT.py b/PRESUBMIT.py index 04ed2e0607..5599c316c3 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -279,10 +279,8 @@ 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, but not include/private. - if (file_ext == '.h' and - 'include' == file_path.split(os.path.sep)[0] and - 'private' not in file_path): + # include dir. + if file_ext == '.h' and 'include' == file_path.split(os.path.sep)[0]: requires_owner_check = True if not requires_owner_check: @@ -330,12 +328,8 @@ def _CheckLGTMsForPublicAPI(input_api, output_api): if not lgtm_from_owner: results.append( output_api.PresubmitError( - "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 your CL saying so " - "and add one of those reviewers on a TBR= line. If you don't know " - "this CL affects Skia's public API, please treat it like it does." - % str(PUBLIC_API_OWNERS))) + 'Since the CL is editing public API, you must have an LGTM from ' + 'one of: %s' % str(PUBLIC_API_OWNERS))) return results |