PRESUBMIT checks’ non-inclusive parameters have been removed

['Josip Sokcevic' via infra-announce]

tl;dr As of today, February 1st 2021, PRESUBMIT checks’ non-inclusive parameters have been removed in favor of more inclusive ones.

Hi all,

As previously shared, the Chrome Operations Source team has removed the non-inclusive parameters in our presubmit API in favor of more inclusively named parameters. Warnings were being issued when the deprecated non-inclusive parameters were still being used since November last year. Additionally, we proactively used Code Search to identify pending clients to migrate and did the changes ourselves.

As a reminder here is the overview of relevant changes (diff input_api, diff canned_checks):

Changes in constant names

input_api.DEFAULT_WHITE_LIST -> input_api.DEFAULT_FILES_TO_CHECK

input_api.DEFAULT_BLACK_LIST -> input_api.DEFAULT_FILES_TO_SKIP

Changes in named parameters

input_api.FilterSourceFile, white_list -> files_to_check, black_list -> files_to_skip

input_api.canned_checks.GetUnitTestsInDirectory, whitelist -> files_to_check, blacklist -> files_to_skip

input_api.canned_checks.GetUnitTestsRecursively, whitelist -> files_to_check, blacklist -> files_to_skip

input_api.canned_checks.GetPylint, white_list -> files_to_check, black_list -> files_to_skip

- Josip and Normando on behalf of the Chops Source team

--
You received this message because you are subscribed to the Google Groups "infra-announce" group.
To unsubscribe from this group and stop receiving emails from it, send an email to infra-announc...@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/infra-announce/CAJiyOiij_xkAwg8upGixR%2BsLq34jCmE5tZeEyyFEYLJRBqDpBQ%40mail.gmail.com.