From f91b717a996553a4ed65b8bf805e56549cd6fde3 Mon Sep 17 00:00:00 2001 From: rmistry Date: Thu, 12 Mar 2015 09:48:10 -0700 Subject: [PATCH] Ignore public API checks if COMMIT=false is in the description BUG=skia: NOTRY=true Committed: https://skia.googlesource.com/skia/+/d1783aefcc0da86c5ff1d124c54704252d817621 Review URL: https://codereview.chromium.org/1006473002 --- PRESUBMIT.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/PRESUBMIT.py b/PRESUBMIT.py index d62249a..ae5a218 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -254,6 +254,11 @@ def _CheckLGTMsForPublicAPI(input_api, output_api): # It is a revert CL, ignore the public api owners check. return results + if re.search(r'^COMMIT=false$', issue_properties['description'], re.M): + # Ignore public api owners check for COMMIT=false CLs since they are not + # going to be committed. + return results + match = re.search(r'^TBR=(.*)$', issue_properties['description'], re.M) if match: tbr_entries = match.group(1).strip().split(',') -- 2.7.4