[PATCH v6 09/29] CI: have "static-analysis" run "check-builtins", not "documentation"
- From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
- Date
- May 25, 2022, 10:03 UTC
- Message-ID
- <patch-v6-09.29-1a9c98b909c-20220525T094123Z-avarab@gmail.com>
- In-Reply-To
- <cover-v6-00.29-00000000000-20220525T094123Z-avarab@gmail.com>
Move the "make check-builtins" check from the "documentation" job to the "static-analysis" job.
The "check-builtins" target added in c74390e4a1d (cherry is built-in, do not ship git-cherry.sh, 2006-11-05) is unrelated to the documentation, so it's odd that b98712b9aa9 (travis-ci: build documentation, 2016-05-04) added it to the "documentation" job.
Let's just move it to the "ci-static-analysis" target, and while we're at it improve the output with $(QUIET_CHECK).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> --- Makefile | 3 ++- ci/test-documentation.sh | 1 - 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile index 9ae7e1a373e..3398d3f2e56 100644 --- a/Makefile +++ b/Makefile @@ -3405,7 +3405,7 @@ check-docs:: ### Make sure built-ins do not have dups and listed in git.c # check-builtins:: - ./check-builtins.sh + $(QUIET_CHECK)./check-builtins.sh ### Test suite coverage testing # @@ -3495,5 +3495,6 @@ ci-check-directional-formatting: .PHONY: ci-static-analysis ci-static-analysis: ci-check-directional-formatting +ci-static-analysis: check-builtins ci-static-analysis: check-coccicheck ci-static-analysis: hdr-check diff --git a/ci/test-documentation.sh b/ci/test-documentation.sh index 9e0652c30dd..41e2b126311 100755 --- a/ci/test-documentation.sh +++ b/ci/test-documentation.sh @@ -14,7 +14,6 @@ filter_log () { "$1" } -make check-builtins make check-docs # Build docs with AsciiDoc
-- 2.36.1.1045.gf356b5617dd