Skip to content

Commit

Permalink
Merge pull request #1124 from NVIDIA/upd-golangci-lint
Browse files Browse the repository at this point in the history
update golangci-lint version to v1.62.0 and pass in version via GITHUB_ENV
  • Loading branch information
tariq1890 authored Nov 20, 2024
2 parents 9378108 + 6187795 commit 4ec60f9
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,9 @@ jobs:
id: vars
run: |
GOLANG_VERSION=$( grep "GOLANG_VERSION ?=" versions.mk )
GOLANGCI_LINT_VERSION=$( grep "GOLANGCI_LINT_VERSION ?=" versions.mk )
echo "GOLANG_VERSION=${GOLANG_VERSION##GOLANG_VERSION ?= }" >> $GITHUB_ENV
echo "GOLANGCI_LINT_VERSION=${GOLANGCI_LINT_VERSION##GOLANGCI_LINT_VERSION ?= }" >> $GITHUB_ENV
- name: Install Go
uses: actions/setup-go@v5
with:
Expand Down Expand Up @@ -82,7 +84,7 @@ jobs:
- name: Lint
uses: golangci/golangci-lint-action@v6
with:
version: v1.60.3
version: ${{ env.GOLANGCI_LINT_VERSION }}
args: -v --timeout 5m
skip-cache: true
- run: make check
Expand Down
2 changes: 2 additions & 0 deletions versions.mk
Original file line number Diff line number Diff line change
Expand Up @@ -21,4 +21,6 @@ VERSION ?= v24.9.0

GOLANG_VERSION ?= 1.23.3

GOLANGCI_LINT_VERSION ?= v1.62.0

GIT_COMMIT ?= $(shell git describe --match="" --dirty --long --always 2> /dev/null || echo "")

0 comments on commit 4ec60f9

Please sign in to comment.