From dbb84c268bd09d2aa47256b5315a87f891583b6b Mon Sep 17 00:00:00 2001 From: Ravi Mistry Date: Wed, 5 Oct 2016 12:47:44 -0400 Subject: 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 Commit-Queue: Ravi Mistry --- PRESUBMIT.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'PRESUBMIT.py') 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 -- cgit v1.2.3