Skip to content

Commit e628d7a

Browse files
committed
chore: groups unconvert files to a package
1 parent 30900e1 commit e628d7a

File tree

4 files changed

+19
-7
lines changed

4 files changed

+19
-7
lines changed

pkg/golinters/unconvert.go renamed to pkg/golinters/unconvert/unconvert.go

+6-6
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package golinters
1+
package unconvert
22

33
import (
44
"sync"
@@ -12,14 +12,14 @@ import (
1212
"github.com/golangci/golangci-lint/pkg/result"
1313
)
1414

15-
const unconvertName = "unconvert"
15+
const name = "unconvert"
1616

17-
func NewUnconvert(settings *config.UnconvertSettings) *goanalysis.Linter {
17+
func New(settings *config.UnconvertSettings) *goanalysis.Linter {
1818
var mu sync.Mutex
1919
var resIssues []goanalysis.Issue
2020

2121
analyzer := &analysis.Analyzer{
22-
Name: unconvertName,
22+
Name: name,
2323
Doc: goanalysis.TheOnlyanalyzerDoc,
2424
Run: func(pass *analysis.Pass) (any, error) {
2525
issues := runUnconvert(pass, settings)
@@ -37,7 +37,7 @@ func NewUnconvert(settings *config.UnconvertSettings) *goanalysis.Linter {
3737
}
3838

3939
return goanalysis.NewLinter(
40-
unconvertName,
40+
name,
4141
"Remove unnecessary type conversions",
4242
[]*analysis.Analyzer{analyzer},
4343
nil,
@@ -54,7 +54,7 @@ func runUnconvert(pass *analysis.Pass, settings *config.UnconvertSettings) []goa
5454
issues = append(issues, goanalysis.NewIssue(&result.Issue{
5555
Pos: position,
5656
Text: "unnecessary conversion",
57-
FromLinter: unconvertName,
57+
FromLinter: name,
5858
}, pass))
5959
}
6060

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
package unconvert
2+
3+
import (
4+
"testing"
5+
6+
"github.com/golangci/golangci-lint/test/testshared/integration"
7+
)
8+
9+
func TestFromTestdata(t *testing.T) {
10+
integration.RunTestdata(t)
11+
}

pkg/lint/lintersdb/builder_linter.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ import (
9999
"github.com/golangci/golangci-lint/pkg/golinters/testpackage"
100100
"github.com/golangci/golangci-lint/pkg/golinters/thelper"
101101
"github.com/golangci/golangci-lint/pkg/golinters/tparallel"
102+
"github.com/golangci/golangci-lint/pkg/golinters/unconvert"
102103
"github.com/golangci/golangci-lint/pkg/golinters/whitespace"
103104
"github.com/golangci/golangci-lint/pkg/golinters/zerologlint"
104105
"github.com/golangci/golangci-lint/pkg/lint/linter"
@@ -748,7 +749,7 @@ func (LinterBuilder) Build(cfg *config.Config) ([]*linter.Config, error) {
748749
WithPresets(linter.PresetBugs).
749750
WithURL(""),
750751

751-
linter.NewConfig(golinters.NewUnconvert(&cfg.LintersSettings.Unconvert)).
752+
linter.NewConfig(unconvert.New(&cfg.LintersSettings.Unconvert)).
752753
WithSince("v1.0.0").
753754
WithLoadForGoAnalysis().
754755
WithPresets(linter.PresetStyle).

0 commit comments

Comments
 (0)