Skip to content

Commit 440133c

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

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
@@ -2888,10 +2888,6 @@ severity:
28882888
# Default: false
28892889
case-sensitive: true
28902890

2891-
# Don't override severity defined by linters.
2892-
# Default: false
2893-
keep-linter-severity: true
2894-
28952891
# When a list of severity rules are provided, severity information will be added to lint issues.
28962892
# Severity rules have the same filtering capability as exclude rules
28972893
# 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
@@ -302,7 +302,6 @@ func getSeverityRulesProcessor(cfg *config.Severity, log logutils.Log, files *fs
302302
Default: cfg.Default,
303303
Rules: severityRules,
304304
CaseSensitive: cfg.CaseSensitive,
305-
Override: !cfg.KeepLinterSeverity,
306305
}
307306

308307
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)