Skip to content

Commit e5051e5

Browse files
committed
chore: groups unparam files to a package
1 parent e628d7a commit e5051e5

File tree

4 files changed

+19
-7
lines changed

4 files changed

+19
-7
lines changed
File renamed without changes.

pkg/golinters/unparam.go renamed to pkg/golinters/unparam/unparam.go

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

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

17-
const unparamName = "unparam"
17+
const name = "unparam"
1818

19-
func NewUnparam(settings *config.UnparamSettings) *goanalysis.Linter {
19+
func New(settings *config.UnparamSettings) *goanalysis.Linter {
2020
var mu sync.Mutex
2121
var resIssues []goanalysis.Issue
2222

2323
analyzer := &analysis.Analyzer{
24-
Name: unparamName,
24+
Name: name,
2525
Doc: goanalysis.TheOnlyanalyzerDoc,
2626
Requires: []*analysis.Analyzer{buildssa.Analyzer},
2727
Run: func(pass *analysis.Pass) (any, error) {
@@ -43,7 +43,7 @@ func NewUnparam(settings *config.UnparamSettings) *goanalysis.Linter {
4343
}
4444

4545
return goanalysis.NewLinter(
46-
unparamName,
46+
name,
4747
"Reports unused function parameters",
4848
[]*analysis.Analyzer{analyzer},
4949
nil,
@@ -82,7 +82,7 @@ func runUnparam(pass *analysis.Pass, settings *config.UnparamSettings) ([]goanal
8282
issues = append(issues, goanalysis.NewIssue(&result.Issue{
8383
Pos: pass.Fset.Position(i.Pos()),
8484
Text: i.Message(),
85-
FromLinter: unparamName,
85+
FromLinter: name,
8686
}, pass))
8787
}
8888

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
package unparam
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
@@ -100,6 +100,7 @@ import (
100100
"github.com/golangci/golangci-lint/pkg/golinters/thelper"
101101
"github.com/golangci/golangci-lint/pkg/golinters/tparallel"
102102
"github.com/golangci/golangci-lint/pkg/golinters/unconvert"
103+
"github.com/golangci/golangci-lint/pkg/golinters/unparam"
103104
"github.com/golangci/golangci-lint/pkg/golinters/whitespace"
104105
"github.com/golangci/golangci-lint/pkg/golinters/zerologlint"
105106
"github.com/golangci/golangci-lint/pkg/lint/linter"
@@ -755,7 +756,7 @@ func (LinterBuilder) Build(cfg *config.Config) ([]*linter.Config, error) {
755756
WithPresets(linter.PresetStyle).
756757
WithURL("https://github.com/mdempsky/unconvert"),
757758

758-
linter.NewConfig(golinters.NewUnparam(&cfg.LintersSettings.Unparam)).
759+
linter.NewConfig(unparam.New(&cfg.LintersSettings.Unparam)).
759760
WithSince("v1.9.0").
760761
WithPresets(linter.PresetUnused).
761762
WithLoadForGoAnalysis().

0 commit comments

Comments
 (0)