From 274aee9cc6dd861ab26d00091b07555c41ff5c99 Mon Sep 17 00:00:00 2001 From: Fernandez Ludovic Date: Tue, 22 Feb 2022 21:49:54 +0100 Subject: [PATCH] revive: fix default values --- pkg/config/linters_settings.go | 4 ++++ pkg/golinters/revive.go | 8 +------- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/pkg/config/linters_settings.go b/pkg/config/linters_settings.go index 36a9af34c752..6b4e4b5ffb5f 100644 --- a/pkg/config/linters_settings.go +++ b/pkg/config/linters_settings.go @@ -79,6 +79,10 @@ var defaultLintersSettings = LintersSettings{ Ignore: "", Qualified: false, }, + Revive: ReviveSettings{ + Confidence: 0.8, + Severity: "warning", + }, Testpackage: TestpackageSettings{ SkipRegexp: `(export|internal)_test\.go`, }, diff --git a/pkg/golinters/revive.go b/pkg/golinters/revive.go index d8165f3cede7..a30993105bc7 100644 --- a/pkg/golinters/revive.go +++ b/pkg/golinters/revive.go @@ -9,7 +9,6 @@ import ( "reflect" "github.com/BurntSushi/toml" - "github.com/mgechev/dots" reviveConfig "github.com/mgechev/revive/config" "github.com/mgechev/revive/lint" "github.com/mgechev/revive/rule" @@ -50,10 +49,10 @@ func NewRevive(cfg *config.ReviveSettings) *goanalysis.Linter { ).WithContextSetter(func(lintCtx *linter.Context) { analyzer.Run = func(pass *analysis.Pass) (interface{}, error) { var files []string - for _, file := range pass.Files { files = append(files, pass.Fset.PositionFor(file.Pos(), false).Filename) } + packages := [][]string{files} conf, err := getReviveConfig(cfg) if err != nil { @@ -72,11 +71,6 @@ func NewRevive(cfg *config.ReviveSettings) *goanalysis.Linter { return nil, err } - packages, err := dots.ResolvePackages(files, []string{}) - if err != nil { - return nil, err - } - failures, err := revive.Lint(packages, lintingRules, *conf) if err != nil { return nil, err