Skip to content

Commit 31123da

Browse files
refactor(config): remove duplicates in enabled rules
1 parent 37faae2 commit 31123da

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

config/lint.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,14 @@ func GetFormatter(conf *lint.Config) (lint.Formatter, error) {
4040
func GetEnabledRules(conf *lint.Config) ([]lint.Rule, error) {
4141
enabledRules := make([]lint.Rule, 0, len(conf.Rules))
4242

43+
// To check if duplicate rule is added
44+
addedRules := make(map[string]struct{})
45+
4346
for _, ruleName := range conf.Rules {
47+
if _, ok := addedRules[ruleName]; ok {
48+
continue
49+
}
50+
4451
// Checking if rule is registered
4552
// before checking if rule is enabled
4653
r, ok := registry.GetRule(ruleName)
@@ -58,6 +65,7 @@ func GetEnabledRules(conf *lint.Config) ([]lint.Rule, error) {
5865
return nil, fmt.Errorf("config error: %v", err)
5966
}
6067
enabledRules = append(enabledRules, r)
68+
addedRules[r.Name()] = struct{}{}
6169
}
6270

6371
return enabledRules, nil

0 commit comments

Comments
 (0)