diff options
author | Ravi Mistry <rmistry@google.com> | 2016-11-23 13:26:34 -0500 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-11-23 18:34:23 +0000 |
commit | 8c6a43a1c06de67311f1fd0d16338f50f4f4c1d6 (patch) | |
tree | 184c23ac91b813889814a58811b8ea54e38d0f24 /PRESUBMIT.py | |
parent | e9d1b299ccbf8017ba57040986ded638f954684a (diff) |
Do not use post upload hooks for 'GOLD_TRYBOT_URL'
That information will be provided via a plugin soon.
# This code is not exercised by trybots.
NOTRY=true
BUG=skia:6000
Change-Id: I6ea60783c0581e32317a781e4349e95b38b71da0
Reviewed-on: https://skia-review.googlesource.com/5207
Commit-Queue: Ravi Mistry <rmistry@google.com>
Reviewed-by: Eric Boren <borenet@google.com>
Diffstat (limited to 'PRESUBMIT.py')
-rw-r--r-- | PRESUBMIT.py | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/PRESUBMIT.py b/PRESUBMIT.py index 4dc21c9fde..a61f732dae 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -448,7 +448,6 @@ def PostUploadHook(cl, change, output_api): """git cl upload will call this hook after the issue is created/modified. This hook does the following: - * Adds a link to the CL's Gold trybot results. * Adds a link to preview docs changes if there are any docs changes in the CL. * Adds 'NOTRY=true' if the CL contains only docs changes. * Adds 'NOTREECHECKS=true' for non master branch changes since they do not @@ -486,15 +485,6 @@ def PostUploadHook(cl, change, output_api): new_description = original_description - # Add GOLD_TRYBOT_URL if it does not exist yet. - if not re.search(r'^GOLD_TRYBOT_URL=', new_description, re.M | re.I): - new_description += '\nGOLD_TRYBOT_URL= %s%s' % (GOLD_TRYBOT_URL, issue) - results.append( - output_api.PresubmitNotifyResult( - 'Added link to Gold trybot runs to the CL\'s description.\n' - 'Note: Results may take sometime to be populated after trybots ' - 'complete.')) - # If the change includes only doc changes then add NOTRY=true in the # CL's description if it does not exist yet. if all_docs_changes and not re.search( |