diff options
author | Ravi Mistry <rmistry@google.com> | 2016-10-05 12:47:44 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-10-05 17:08:55 +0000 |
commit | dbb84c268bd09d2aa47256b5315a87f891583b6b (patch) | |
tree | c9c635f528a9af767675d1c9ae53a51ad940b97c /PRESUBMIT.py | |
parent | 26bea5d55735367152378bae14453e9666d1c625 (diff) |
Fix public API msg in PRESUBMIT.py
BUG=skia:5825
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2989
Change-Id: Ia074ab00ccd1500bee10a5b313636581e8265461
Reviewed-on: https://skia-review.googlesource.com/2989
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>
Diffstat (limited to 'PRESUBMIT.py')
-rw-r--r-- | PRESUBMIT.py | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/PRESUBMIT.py b/PRESUBMIT.py index 320e63d03a..97c54d20ea 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -454,10 +454,11 @@ def _CheckLGTMsForPublicAPI(input_api, output_api): 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 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))) + "Skia's public API, please add a short note to the CL saying so. " + "Add one of the owners as a reviewer to your CL. For Rietveld CLs " + "you also need to add one of those owners 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 |