Revert "Warn when non-inclusive parameters are used"
This reverts commit 149a3b7581257a60a3541e91875893088d68fb7f.
Reason for revert: warnings are issued if no optional parameters
are passed.
Original change's description:
> Warn when non-inclusive parameters are used
>
> R=apolito@google.com
>
> Bug: 1098560
> Change-Id: Iabc2a5d89a6af1418c1228f3c09eacbbd799b5c7
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2355112
> Reviewed-by: Anthony Polito <apolito@google.com>
> Commit-Queue: Josip Sokcevic <sokcevic@google.com>
TBR=apolito@google.com,infra-scoped@luci-project-accounts.iam.gserviceaccount.com,sokcevic@google.com
# Not skipping CQ checks because original CL landed > 1 day ago.
Bug: 1098560
Change-Id: Ib0f188de90121c692f5c0d7271938ee312917fd1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2376042
Reviewed-by: Josip Sokcevic <sokcevic@google.com>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index 7c68cab..d0cc1ba 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -91,8 +91,8 @@
input_api,
output_api,
'tests',
- files_to_check=test_to_run_list,
- files_to_skip=tests_to_skip_list,
+ allowlist=test_to_run_list,
+ blocklist=tests_to_skip_list,
run_on_python3=run_on_python3))
# Validate CIPD manifests.
diff --git a/presubmit_canned_checks.py b/presubmit_canned_checks.py
index 5b41bb7..59a3238 100644
--- a/presubmit_canned_checks.py
+++ b/presubmit_canned_checks.py
@@ -7,7 +7,6 @@
from __future__ import print_function
import os as _os
-from warnings import warn
_HERE = _os.path.dirname(_os.path.abspath(__file__))
# These filters will be disabled if callers do not explicitly supply a
@@ -109,9 +108,8 @@
"""For non-googler/chromites committers, verify the author's email address is
in AUTHORS.
"""
- # TODO(https://crbug.com/1098560): Remove non inclusive parameter names.
+ # TODO(https://crbug.com/1098560): Add warnings before removing BC.
if bot_allowlist is None:
- warn('Use bot_allowlist in CheckAuthorizedAuthor')
bot_allowlist = bot_whitelist
if input_api.is_committing:
error_type = output_api.PresubmitError
@@ -626,12 +624,10 @@
It's mainly a wrapper for RunUnitTests. Use allowlist and blocklist to filter
tests accordingly.
"""
- # TODO(https://crbug.com/1098560): Remove non inclusive parameter names.
+ # TODO(https://crbug.com/1098560): Add warnings before removing bc.
if files_to_check is None:
- warn('Use files_to_check in GetUnitTestsInDirectory')
files_to_check = allowlist or whitelist
if files_to_skip is None:
- warn('Use files_to_skip in GetUnitTestsInDirectory')
files_to_skip = blocklist or blacklist
unit_tests = []
@@ -726,12 +722,10 @@
Restricts itself to only find files within the Change's source repo, not
dependencies.
"""
- # TODO(https://crbug.com/1098560): Remove non inclusive parameter names.
+ # TODO(https://crbug.com/1098560): Add warnings before removing BC.
if files_to_check is None:
- warn('Use files_to_check in GetUnitTestsRecursively')
files_to_check = allowlist or whitelist
if files_to_skip is None:
- warn('Use files_to_skip in GetUnitTestsRecursively')
files_to_skip = blocklist or blacklist
assert files_to_check is not None
assert files_to_skip is not None
@@ -876,13 +870,6 @@
The default files_to_check enforces looking only at *.py files.
"""
-
- # TODO(https://crbug.com/1098560): Remove non inclusive parameter names.
- if allow_list or white_list:
- warn('Use files_to_check in GetPylint')
- if block_list or black_list:
- warn('Use files_to_skip in GetPylint')
-
files_to_check = tuple(files_to_check or allow_list or white_list or
(r'.*\.py$',))
files_to_skip = tuple(files_to_skip or block_list or black_list or
diff --git a/presubmit_support.py b/presubmit_support.py
index ee2883c..53ed9c5 100755
--- a/presubmit_support.py
+++ b/presubmit_support.py
@@ -730,12 +730,10 @@
Note: Copy-paste this function to suit your needs or use a lambda function.
"""
- # TODO(https://crbug.com/1098560): Remove non inclusive parameter names.
- if files_to_check is None and (allow_list or white_list):
- warn('Use files_to_check in FilterSourceFile')
+ # TODO(https://crbug.com/1098560): Add warnings before removing bc.
+ if files_to_check is None:
files_to_check = allow_list or white_list
- if files_to_skip is None and (block_list or black_list):
- warn('Use files_to_skip in FilterSourceFile')
+ if files_to_skip is None:
files_to_skip = block_list or black_list
if files_to_check is None: