Commit e9bc2113 authored by James Cook's avatar James Cook Committed by Commit Bot

mojo: Use inclusive language in PRESUBMIT.py

chops recently updated the depot_tools input_api, which makes this
possible.

input_api.DEFAULT_ALLOW_LIST -> input_api.DEFAULT_FILES_TO_CHECK
input_api.DEFAULT_BLACK_LIST -> input_api.DEFAULT_FILES_TO_SKIP

For named parameters:
white_list/whitelist -> files_to_check
black_list/blacklist -> files_to_skip

Bug: 1098562
Change-Id: Icc576b50727256d94ad62b985b2c81991a9af49e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317743
Auto-Submit: James Cook <jamescook@chromium.org>
Commit-Queue: Ken Rockot <rockot@google.com>
Reviewed-by: default avatarKen Rockot <rockot@google.com>
Cr-Commit-Position: refs/heads/master@{#791975}
parent 89f4148d
......@@ -24,7 +24,7 @@ def CheckChangeOnUpload(input_api, output_api):
input_api.PresubmitLocalPath(), "public", "python")
# TODO(vtl): Don't lint these files until the (many) problems are fixed
# (possibly by deleting/rewriting some files).
temporary_black_list = input_api.DEFAULT_BLACK_LIST + \
files_to_skip = input_api.DEFAULT_FILES_TO_SKIP + \
(r".*\bpublic[\\\/]tools[\\\/]bindings[\\\/]pylib[\\\/]mojom[\\\/]"
r"generate[\\\/].+\.py$",
r".*\bpublic[\\\/]tools[\\\/]bindings[\\\/]generators[\\\/].+\.py$",
......@@ -40,5 +40,5 @@ def CheckChangeOnUpload(input_api, output_api):
]
results += input_api.canned_checks.RunPylint(
input_api, output_api, extra_paths_list=pylint_extra_paths,
black_list=temporary_black_list)
files_to_skip=files_to_skip)
return results
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment