diff --git a/PRESUBMIT.py b/PRESUBMIT.py index 13a2428fe..8c4de46e0 100644 --- a/PRESUBMIT.py +++ b/PRESUBMIT.py @@ -56,6 +56,8 @@ def DepotToolsPylint(input_api, output_api): def CommonChecks(input_api, output_api, tests_to_black_list): results = [] results.extend(input_api.canned_checks.CheckOwners(input_api, output_api)) + results.extend(input_api.canned_checks.CheckOwnersFormat( + input_api, output_api)) # TODO(maruel): Make sure at least one file is modified first. # TODO(maruel): If only tests are modified, only run them. tests = DepotToolsPylint(input_api, output_api) diff --git a/presubmit_canned_checks.py b/presubmit_canned_checks.py index 710a8d966..584a70801 100644 --- a/presubmit_canned_checks.py +++ b/presubmit_canned_checks.py @@ -1013,6 +1013,10 @@ def PanProjectChecks(input_api, output_api, print " %s took a long time: %dms" % (snapshot_memory[1], delta_ms) snapshot_memory[:] = (dt2, msg) + snapshot("checking owners files format") + results.extend(input_api.canned_checks.CheckOwnersFormat( + input_api, output_api)) + if owners_check: snapshot("checking owners") results.extend(input_api.canned_checks.CheckOwners(