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

Preserve API backward compatibility

Browse Source
This commit is contained in:
John Starich 2020-04-25 14:14:42 -05:00
parent 15c75690fc
commit 36d8d881f9
3 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
pkg/config/config.go
View File

@ -101,7 +101,11 @@ var DefaultExcludePatterns = []ExcludePattern{
}, },
} }
func GetDefaultExcludePatternsStrings(include []string) []string { func GetDefaultExcludePatternsStrings() []string {
return GetExcludePatternsStrings(nil)
}
func GetExcludePatternsStrings(include []string) []string {
includeMap := make(map[string]bool, len(include)) includeMap := make(map[string]bool, len(include))
for _, inc := range include { for _, inc := range include {
includeMap[inc] = true includeMap[inc] = true

2
pkg/lint/runner.go
View File

@ -32,7 +32,7 @@ func NewRunner(cfg *config.Config, log logutils.Log, goenv *goutil.Env,
icfg := cfg.Issues icfg := cfg.Issues
excludePatterns := icfg.ExcludePatterns excludePatterns := icfg.ExcludePatterns
if icfg.UseDefaultExcludes { if icfg.UseDefaultExcludes {
excludePatterns = append(excludePatterns, config.GetDefaultExcludePatternsStrings(icfg.IncludeDefaultExcludes)...) excludePatterns = append(excludePatterns, config.GetExcludePatternsStrings(icfg.IncludeDefaultExcludes)...)
} }
var excludeTotalPattern string var excludeTotalPattern string

2
test/bench/bench_test.go
View File

@ -77,7 +77,7 @@ func getGometalinterCommonArgs() []string {
"--vendor", "--vendor",
"--cyclo-over=30", "--cyclo-over=30",
"--dupl-threshold=150", "--dupl-threshold=150",
"--exclude", fmt.Sprintf("(%s)", strings.Join(config.GetDefaultExcludePatternsStrings(nil), "|")), "--exclude", fmt.Sprintf("(%s)", strings.Join(config.GetDefaultExcludePatternsStrings(), "|")),
"--disable-all", "--disable-all",
"--enable=vet", "--enable=vet",
"--enable=vetshadow", "--enable=vetshadow",