Skip to content

Commit cdea24f

Browse files
committed
Revert "feat: option to not override severity from linters (golangci#4452)"
This reverts commit 3d91352.
1 parent 1cabafc commit cdea24f

File tree

4 files changed

+3
-16
lines changed

4 files changed

+3
-16
lines changed

.golangci.reference.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2883,10 +2883,6 @@ severity:
28832883
# Default: false
28842884
case-sensitive: true
28852885

2886-
# Don't override severity defined by linters.
2887-
# Default: false
2888-
keep-linter-severity: true
2889-
28902886
# When a list of severity rules are provided, severity information will be added to lint issues.
28912887
# Severity rules have the same filtering capability as exclude rules
28922888
# except you are allowed to specify one matcher per severity rule.

pkg/config/severity.go

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,9 @@ import (
88
const severityRuleMinConditionsCount = 1
99

1010
type Severity struct {
11-
Default string `mapstructure:"default-severity"`
12-
CaseSensitive bool `mapstructure:"case-sensitive"`
13-
Rules []SeverityRule `mapstructure:"rules"`
14-
KeepLinterSeverity bool `mapstructure:"keep-linter-severity"` // TODO(ldez): in v2 should be changed to `Override`.
11+
Default string `mapstructure:"default-severity"`
12+
CaseSensitive bool `mapstructure:"case-sensitive"`
13+
Rules []SeverityRule `mapstructure:"rules"`
1514
}
1615

1716
func (s *Severity) Validate() error {

pkg/lint/runner.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,6 @@ func getSeverityRulesProcessor(cfg *config.Severity, log logutils.Log, files *fs
317317
Default: cfg.Default,
318318
Rules: severityRules,
319319
CaseSensitive: cfg.CaseSensitive,
320-
Override: !cfg.KeepLinterSeverity,
321320
}
322321

323322
return processors.NewSeverity(log.Child(logutils.DebugKeySeverityRules), files, severityOpts)

pkg/result/processors/severity.go

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ type SeverityOptions struct {
2424
Default string
2525
Rules []SeverityRule
2626
CaseSensitive bool
27-
Override bool
2827
}
2928

3029
type Severity struct {
@@ -36,7 +35,6 @@ type Severity struct {
3635

3736
defaultSeverity string
3837
rules []severityRule
39-
override bool
4038
}
4139

4240
func NewSeverity(log logutils.Log, files *fsutils.Files, opts SeverityOptions) *Severity {
@@ -45,7 +43,6 @@ func NewSeverity(log logutils.Log, files *fsutils.Files, opts SeverityOptions) *
4543
files: files,
4644
log: log,
4745
defaultSeverity: opts.Default,
48-
override: opts.Override,
4946
}
5047

5148
prefix := caseInsensitivePrefix
@@ -65,10 +62,6 @@ func (p *Severity) Process(issues []result.Issue) ([]result.Issue, error) {
6562
}
6663

6764
return transformIssues(issues, func(issue *result.Issue) *result.Issue {
68-
if issue.Severity != "" && !p.override {
69-
return issue
70-
}
71-
7265
for _, rule := range p.rules {
7366
rule := rule
7467

0 commit comments

Comments
 (0)