From 3d2336bc83e85f009ca2c4de8fb159a9138119ca Mon Sep 17 00:00:00 2001 From: Michael Date: Wed, 23 Oct 2024 17:06:04 +0200 Subject: [PATCH] Use golangci-lint action --- .github/workflows/validate.yaml | 24 ++++++++++++++++++++---- Makefile | 10 +++------- 2 files changed, 23 insertions(+), 11 deletions(-) diff --git a/.github/workflows/validate.yaml b/.github/workflows/validate.yaml index 2ccec8f24..abcbc97d7 100644 --- a/.github/workflows/validate.yaml +++ b/.github/workflows/validate.yaml @@ -12,6 +12,25 @@ env: jobs: + lint: + runs-on: ubuntu-latest + + steps: + - name: Check out code + uses: actions/checkout@v4 + with: + fetch-depth: 0 + + - name: Set up Go ${{ env.GO_VERSION }} + uses: actions/setup-go@v5 + with: + go-version: ${{ env.GO_VERSION }} + + - name: golangci-lint + uses: golangci/golangci-lint-action@v6 + with: + version: "${{ env.GOLANGCI_LINT_VERSION }}" + validate: runs-on: ubuntu-latest @@ -26,9 +45,6 @@ jobs: with: go-version: ${{ env.GO_VERSION }} - - name: Install golangci-lint ${{ env.GOLANGCI_LINT_VERSION }} - run: curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin ${GOLANGCI_LINT_VERSION} - - name: Install misspell ${{ env.MISSPELL_VERSION }} run: curl -sfL https://raw.githubusercontent.com/golangci/misspell/master/install-misspell.sh | sh -s -- -b $(go env GOPATH)/bin ${MISSPELL_VERSION} @@ -36,7 +52,7 @@ jobs: run: touch webui/static/index.html - name: Validate - run: make validate + run: make validate-files validate-generate: runs-on: ubuntu-latest diff --git a/Makefile b/Makefile index af62ff77c..393d1b62a 100644 --- a/Makefile +++ b/Makefile @@ -120,20 +120,16 @@ lint: .PHONY: validate-files #? validate-files: Validate code and docs -validate-files: lint +validate-files: $(foreach exec,$(LINT_EXECUTABLES),\ $(if $(shell which $(exec)),,$(error "No $(exec) in PATH"))) + $(CURDIR)/script/validate-vendor.sh $(CURDIR)/script/validate-misspell.sh $(CURDIR)/script/validate-shell-script.sh .PHONY: validate #? validate: Validate code, docs, and vendor -validate: lint - $(foreach exec,$(EXECUTABLES),\ - $(if $(shell which $(exec)),,$(error "No $(exec) in PATH"))) - $(CURDIR)/script/validate-vendor.sh - $(CURDIR)/script/validate-misspell.sh - $(CURDIR)/script/validate-shell-script.sh +validate: lint validate-files # Target for building images for multiple architectures. .PHONY: multi-arch-image-%