Skip to content

Commit

Permalink
[PRESUBMIT] Update _NON_INCLUSIVE_TERMS regex
Browse files Browse the repository at this point in the history
The old regex is not python 3.11 compatible.

R=stevensd@google.com

(cherry picked from commit 9d2806a)

Fixed: 1511105
Change-Id: I95cf24490328cc53f85acc1d350261f8e312f919
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5118881
Auto-Submit: Josip Sokcevic <sokcevic@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Original-Commit-Position: refs/heads/main@{#1236755}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5170553
Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
Cr-Commit-Position: refs/branch-heads/5993@{#1711}
Cr-Branched-From: 5113507-refs/heads/main@{#1192594}
  • Loading branch information
sokcevicG authored and Chromium LUCI CQ committed Jan 11, 2024
1 parent ae22a3a commit 5018382
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion PRESUBMIT.py
Original file line number Diff line number Diff line change
Expand Up @@ -5450,7 +5450,7 @@ def CheckEsLintConfigChanges(input_api, output_api):
# ...' will not. This may require some tweaking to catch these cases
# without triggering a lot of false positives. Leaving it naive and
# less matchy for now.
r'/\b(?i)((black|white)list|master|slave)\b', # nocheck
r'/(?i)\b((black|white)list|master|slave)\b', # nocheck
(
'Please don\'t use blacklist, whitelist, ' # nocheck
'or slave in your', # nocheck
Expand Down

0 comments on commit 5018382

Please sign in to comment.