eeafj/golangci-lint
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 1 Wiki Activity

Fix after #890: don't propagate linter errors for golangci.com

Browse Source
This commit is contained in:
Denis Isaev 2020-01-19 16:19:23 +03:00
parent 1ac15487b2
commit 40d7bcd53e
No known key found for this signature in database
GPG Key ID: A36A0EC8E27A1A01
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pkg/lint/runner.go
View File

@ -3,6 +3,7 @@ package lint
import ( import (
"context" "context"
"fmt" "fmt"
"os"
"runtime/debug" "runtime/debug"
"strings" "strings"
@ -187,7 +188,10 @@ func (r Runner) Run(ctx context.Context, linters []*linter.Config, lintCtx *lint
linterIssues, err := r.runLinterSafe(ctx, lintCtx, lc) linterIssues, err := r.runLinterSafe(ctx, lintCtx, lc)
if err != nil { if err != nil {
r.Log.Warnf("Can't run linter %s: %s", lc.Linter.Name(), err) r.Log.Warnf("Can't run linter %s: %s", lc.Linter.Name(), err)
if os.Getenv("GOLANGCI_COM_RUN") == "" {
// Don't stop all linters on one linter failure for golangci.com.
runErr = err runErr = err
}
return return
} }
issues = append(issues, linterIssues...) issues = append(issues, linterIssues...)