From 04abf130bb15c5cdaa2d895ef5b39698a7fb2419 Mon Sep 17 00:00:00 2001 From: rmistry Date: Tue, 7 Apr 2015 07:41:51 -0700 Subject: Bypass owners check for dry run CLs. BUG=skia: NOTRY=true Review URL: https://codereview.chromium.org/1068853002 --- PRESUBMIT.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'PRESUBMIT.py') diff --git a/PRESUBMIT.py b/PRESUBMIT.py index a036be6fc8..60439d6eb0 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -283,8 +283,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 + # TODO(rmistry): Stop checking for COMMIT=false once crbug/470609 is + # resolved. + if issue_properties['cq_dry_run'] or re.search( + r'^COMMIT=false$', issue_properties['description'], re.M): + # Ignore public api owners check for dry run CLs since they are not # going to be committed. return results -- cgit v1.2.3