[PATCH] CI: Run scripts/check_maintainers_files.sh in the existing MAINTAINERS job
Tom Rini <[email protected]>
| Newsgroups | gmane.comp.boot-loaders.u-boot |
|---|---|
| Message-ID | <20260821203657.3946004-1-trini__5952.75848624937$1787344638$gmane$org@konsulko.com> |
Rename the existing job that does some checking of MAINTAINERS file contents already to be a little clearer that it's doing two things now, and then also run scripts/check_maintainers_files.sh in this job. Suggested-by: Michal Simek <[email protected]> Signed-off-by: Tom Rini <[email protected]> --- The overhead any CI system has on spinning up another worker means that it's noticeably more time consuming to make this job standalone than it is to repurpose an existing job. --- .azure-pipelines.yml | 3 ++- .gitlab-ci.yml | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml index 015fc15d5c29..5130058925d5 100644 --- a/.azure-pipelines.yml +++ b/.azure-pipelines.yml @@ -90,7 +90,7 @@ stages: make infodocs - job: maintainers - displayName: 'Ensure all configs have MAINTAINERS entries' + displayName: 'Check all configs are in MAINTAINERS and validate MAINTAINERS' pool: vmImage: $(ubuntu_vm) container: @@ -99,6 +99,7 @@ stages: steps: - script: | ./tools/buildman/buildman --maintainer-check + ./scripts/check_maintainers_files.sh - job: allyesconfig_and_tools displayName: 'Ensure allyesconfig, tools-only and envtools build' diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 9831dd61bc18..e08068f1cbf5 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -189,11 +189,11 @@ docs: - make htmldocs KDOC_WERROR=1 - make infodocs -# ensure all configs have MAINTAINERS entries -Check for configs without MAINTAINERS entry: +Check all configs are in MAINTAINERS and validate MAINTAINERS: extends: .testsuites script: - ./tools/buildman/buildman --maintainer-check + - ./scripts/check_maintainers_files.sh # Ensure host tools build Build allyesconfig, tools-only and envtools: -- 2.43.0