diff options
author | rmistry <rmistry@google.com> | 2015-04-07 07:41:51 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-04-07 07:41:51 -0700 |
commit | 04abf130bb15c5cdaa2d895ef5b39698a7fb2419 (patch) | |
tree | ece60b6a5509a0905617a4145ad00bd74fdbf435 /PRESUBMIT.py | |
parent | e3bea5cc609a82411edd98b0a3f0b542f65ba67e (diff) |
Bypass owners check for dry run CLs.
BUG=skia:
NOTRY=true
Review URL: https://codereview.chromium.org/1068853002
Diffstat (limited to 'PRESUBMIT.py')
-rw-r--r-- | PRESUBMIT.py | 7 |
1 files changed, 5 insertions, 2 deletions
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 |