From 23013d3f68f509d76e882bdaee711c93ecc62dfd Mon Sep 17 00:00:00 2001 From: Alexander Melentyev Date: Fri, 1 Oct 2021 16:05:19 +0300 Subject: [PATCH] Add golangci-lint --- .github/workflows/ci.yml | 6 ++++++ .golangci.yml | 12 ++++++++++++ lint | 26 -------------------------- 3 files changed, 18 insertions(+), 26 deletions(-) create mode 100644 .golangci.yml delete mode 100755 lint diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 9f5c2c79d..f2040437b 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -52,6 +52,12 @@ jobs: - name: Checkout code uses: actions/checkout@v2 + + - name: Run linter + uses: golangci/golangci-lint-action@v2 + with: + version: v1.41 + - name: Run tests env: diff --git a/.golangci.yml b/.golangci.yml new file mode 100644 index 000000000..07c86eaa9 --- /dev/null +++ b/.golangci.yml @@ -0,0 +1,12 @@ +linters: + disable-all: true + enable: + - deadcode + - dupl + - gocritic + - gosec + - gosimple + - structcheck + - typecheck + - unused + - varcheck diff --git a/lint b/lint deleted file mode 100755 index 12e307273..000000000 --- a/lint +++ /dev/null @@ -1,26 +0,0 @@ -#!/usr/bin/env bash - -set -o errexit -set -o nounset -set -o pipefail - -if [ ! $(command -v gometalinter) ] -then - go get github.com/alecthomas/gometalinter - gometalinter --update --install -fi - -time gometalinter \ - --exclude='error return value not checked.*(Close|Log|Print).*\(errcheck\)$' \ - --exclude='.*_test\.go:.*error return value not checked.*\(errcheck\)$' \ - --exclude='/thrift/' \ - --exclude='/pb/' \ - --exclude='no args in Log call \(vet\)' \ - --disable=dupl \ - --disable=aligncheck \ - --disable=gotype \ - --cyclo-over=20 \ - --tests \ - --concurrency=2 \ - --deadline=300s \ - ./...