improved warnings logging

Author: jirfag

pkg/commands/run_config.go

@@ -9,6 +9,7 @@ import (
 
 	"github.com/golangci/golangci-lint/pkg/config"
 	"github.com/golangci/golangci-lint/pkg/fsutils"
+	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/printers"
 	"github.com/sirupsen/logrus"
 	"github.com/spf13/pflag"
@@ -83,7 +84,7 @@ func setupConfigFileSearch(args []string) {
 
 	absStartPath, err := filepath.Abs(firstArg)
 	if err != nil {
-		logrus.Infof("Can't make abs path for %q: %s", firstArg, err)
+		logutils.HiddenWarnf("Can't make abs path for %q: %s", firstArg, err)
 		absStartPath = filepath.Clean(firstArg)
 	}
 
@@ -116,13 +117,13 @@ func setupConfigFileSearch(args []string) {
 func getRelPath(p string) string {
 	wd, err := os.Getwd()
 	if err != nil {
-		logrus.Infof("Can't get wd: %s", err)
+		logutils.HiddenWarnf("Can't get wd: %s", err)
 		return p
 	}
 
 	r, err := filepath.Rel(wd, p)
 	if err != nil {
-		logrus.Infof("Can't make path %s relative to %s: %s", p, wd, err)
+		logutils.HiddenWarnf("Can't make path %s relative to %s: %s", p, wd, err)
 		return p
 	}
 

pkg/golinters/gas.go

@@ -11,8 +11,8 @@ import (
 	"github.com/GoASTScanner/gas"
 	"github.com/GoASTScanner/gas/rules"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
+	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/result"
-	"github.com/sirupsen/logrus"
 )
 
 type Gas struct{}
@@ -45,7 +45,7 @@ func (lint Gas) Run(ctx context.Context, lintCtx *linter.Context) ([]result.Issu
 		line, err := strconv.Atoi(i.Line)
 		if err != nil {
 			if n, rerr := fmt.Sscanf(i.Line, "%d-%d", &r.From, &r.To); rerr != nil || n != 2 {
-				logrus.Infof("Can't convert gas line number %q of %v to int: %s", i.Line, i, err)
+				logutils.HiddenWarnf("Can't convert gas line number %q of %v to int: %s", i.Line, i, err)
 				continue
 			}
 			line = r.From

pkg/golinters/golint.go

@@ -7,8 +7,8 @@ import (
 
 	lintAPI "github.com/golang/lint"
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
+	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/result"
-	"github.com/sirupsen/logrus"
 )
 
 type Golint struct{}
@@ -33,7 +33,7 @@ func (g Golint) Run(ctx context.Context, lintCtx *linter.Context) ([]result.Issu
 		issues = append(issues, i...)
 	}
 	if lintErr != nil {
-		logrus.Infof("golint: %s", lintErr)
+		logutils.HiddenWarnf("golint: %s", lintErr)
 	}
 
 	return issues, nil

pkg/lint/runner.go

@@ -10,6 +10,7 @@ import (
 	"time"
 
 	"github.com/golangci/golangci-lint/pkg/lint/linter"
+	"github.com/golangci/golangci-lint/pkg/logutils"
 	"github.com/golangci/golangci-lint/pkg/result"
 	"github.com/golangci/golangci-lint/pkg/result/processors"
 	"github.com/golangci/golangci-lint/pkg/timeutils"
@@ -30,7 +31,7 @@ func runLinterSafe(ctx context.Context, lintCtx *linter.Context, lc linter.Confi
 	defer func() {
 		if panicData := recover(); panicData != nil {
 			err = fmt.Errorf("panic occured: %s", panicData)
-			logrus.Infof("Panic stack trace: %s", debug.Stack())
+			logutils.HiddenWarnf("Panic stack trace: %s", debug.Stack())
 		}
 	}()
 
@@ -151,7 +152,7 @@ func (r SimpleRunner) processLintResults(ctx context.Context, inCh <-chan lintRe
 
 		for res := range inCh {
 			if res.err != nil {
-				logrus.Infof("Can't run linter %s: %s", res.linter.Linter.Name(), res.err)
+				logutils.HiddenWarnf("Can't run linter %s: %s", res.linter.Linter.Name(), res.err)
 				continue
 			}
 
@@ -220,7 +221,7 @@ func (r *SimpleRunner) processIssues(ctx context.Context, issues []result.Issue,
 		})
 
 		if err != nil {
-			logrus.Infof("Can't process result by %s processor: %s", p.Name(), err)
+			logutils.HiddenWarnf("Can't process result by %s processor: %s", p.Name(), err)
 		} else {
 			issues = newIssues
 		}

pkg/logutils/logutils.go

@@ -0,0 +1,17 @@
+package logutils
+
+import (
+	"os"
+
+	"github.com/sirupsen/logrus"
+)
+
+var isGolangCIRun = os.Getenv("GOLANGCI_COM_RUN") == "1"
+
+func HiddenWarnf(format string, args ...interface{}) {
+	if isGolangCIRun {
+		logrus.Warnf(format, args...)
+	} else {
+		logrus.Infof(format, args...)
+	}
+}