From def70012b9dd55dedd5255958eb74dee7f08ef06 Mon Sep 17 00:00:00 2001 From: Chris Evich Date: Mon, 5 Jun 2023 12:41:39 -0400 Subject: [PATCH] Limit git-validation to 'short-subject' This tool sometimes throws nonsensical or difficult to debug errors. Verifying DCO and white-space issues in commits has long since been moved to other tools (github-actions and git pre-commit hooks). There's no need to duplicate these checks with the git-validation tool. Signed-off-by: Chris Evich --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 948751f1a6..89104f6760 100644 --- a/Makefile +++ b/Makefile @@ -258,7 +258,7 @@ help: ## (Default) Print listing of key targets with their descriptions .PHONY: .gitvalidation .gitvalidation: @echo "Validating vs commit '$(call err_if_empty,EPOCH_TEST_COMMIT)'" - GIT_CHECK_EXCLUDE="./vendor:./test/tools/vendor:docs/make.bat:test/buildah-bud/buildah-tests.diff:test/e2e/quadlet/remap-keep-id2.container" ./test/tools/build/git-validation -run DCO,short-subject,dangling-whitespace -range $(EPOCH_TEST_COMMIT)..$(HEAD) + GIT_CHECK_EXCLUDE="./vendor:./test/tools/vendor:docs/make.bat:test/buildah-bud/buildah-tests.diff:test/e2e/quadlet/remap-keep-id2.container" ./test/tools/build/git-validation -run short-subject -range $(EPOCH_TEST_COMMIT)..$(HEAD) .PHONY: lint lint: golangci-lint