Remove CheckChangeHasTestField PRESUBMIT check.
Depot tools removed it in https://chromium-review.googlesource.com/c/506450/ so we don't have a choice. We never enforced it or had it connected to any organized follow-up anyway, so it makes sense to remove it. BUG=None TBR=ehmaldonado@webrtc.org NOTRY=True Review-Url: https://codereview.webrtc.org/2892873002 Cr-Commit-Position: refs/heads/master@{#18205}
This commit is contained in:
@ -598,8 +598,6 @@ def CheckChangeOnCommit(input_api, output_api):
|
||||
results.extend(input_api.canned_checks.CheckChangeHasDescription(
|
||||
input_api, output_api))
|
||||
results.extend(_CheckChangeHasBugField(input_api, output_api))
|
||||
results.extend(input_api.canned_checks.CheckChangeHasTestField(
|
||||
input_api, output_api))
|
||||
results.extend(input_api.canned_checks.CheckTreeIsOpen(
|
||||
input_api, output_api,
|
||||
json_url='http://webrtc-status.appspot.com/current?format=json'))
|
||||
|
Reference in New Issue
Block a user