From 694b1565d148f33c20f84013fff77fcd9605f56d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Miloslav=20Trma=C4=8D?= Date: Wed, 5 Apr 2023 20:03:38 +0200 Subject: [PATCH] Lint many more files in validate-lint MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Always lint everything, not just changed files; that means that if we upgrade the linter, we will need to clean everything up, but that's a good thing for contributors who come after that linter upgrade. - Don't skip linting the integration tests, there's no good reason to skip them. Signed-off-by: Miloslav Trmač --- hack/make/validate-lint | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/hack/make/validate-lint b/hack/make/validate-lint index 3674d723..a59b28c1 100755 --- a/hack/make/validate-lint +++ b/hack/make/validate-lint @@ -1,12 +1,7 @@ #!/bin/bash -source "$(dirname "$BASH_SOURCE")/.validate" - -# We will eventually get to the point where packages should be the complete list -# of subpackages, vendoring excluded, as given by: -# IFS=$'\n' -files=( $(validate_diff --diff-filter=ACMR --name-only -- '*.go' | grep -v '^vendor/\|^integration' || true) ) +files=( $(find . -name '*.go' | grep -v '^\./vendor/' | sort || true) ) unset IFS errors=()