Ignore exported

Assume disabled by default
golangci · Feb 11, 2021 · f0dd9bc · f0dd9bc
1 parent a35a96f
commit f0dd9bc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/.golangci.yml b/.golangci.yml
@@ -117,6 +117,7 @@ linters:
   # - nestif
   # - prealloc
   # - testpackage
+  # - revive
   # - wsl
 
 issues:

diff --git a/pkg/golinters/revive.go b/pkg/golinters/revive.go
@@ -160,6 +160,10 @@ func SetReviveConfig(cfg *config.ReviveSettings) (*lint.Config, error) {
 		conf.Confidence = cfg.Confidence
 	}
 
+	// By default golangci-lint ignores missing doc comments, follow same convention by removing this default rule
+	// Relevant issue: https://github.com/golangci/golangci-lint/issues/456
+	delete(conf.Rules, "exported")
+
 	if len(cfg.Rules) != 0 {
 		// Clear default rules, only use rules defined in config
 		conf.Rules = map[string]lint.RuleConfig{}