From 68f514fc38ecfcd51efcb0bd03e052ec1c7d31be Mon Sep 17 00:00:00 2001 From: Fabio Baltieri Date: Tue, 25 Jul 2023 18:10:03 +0000 Subject: [PATCH] scripts: compliance: always run the MaintainersFormat check The check currently only runs if the maintainers file itself is changed, but that means that the check is going to miss every PR that moves directory or delete files that can potentially trigger an error. This check is cheap to run, just run it unconditionally. Signed-off-by: Fabio Baltieri --- scripts/ci/check_compliance.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/ci/check_compliance.py b/scripts/ci/check_compliance.py index bf96755ecf..215d91203f 100755 --- a/scripts/ci/check_compliance.py +++ b/scripts/ci/check_compliance.py @@ -1073,8 +1073,8 @@ class MaintainersFormat(ComplianceTest): def run(self): MAINTAINERS_FILES = ["MAINTAINERS.yml", "MAINTAINERS.yaml"] - for file in get_files(filter="d"): - if file not in MAINTAINERS_FILES: + for file in MAINTAINERS_FILES: + if not os.path.exists(file): continue try: