Fix public API msg in PRESUBMIT.py
authorRavi Mistry <rmistry@google.com>
Wed, 5 Oct 2016 16:47:44 +0000 (12:47 -0400)
committerSkia Commit-Bot <skia-commit-bot@chromium.org>
Wed, 5 Oct 2016 17:08:55 +0000 (17:08 +0000)
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>

PRESUBMIT.py

index 320e63d..97c54d2 100644 (file)
@@ -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