Closed zak-pawel closed 2 weeks ago
I think we definitively need to exclude strconv.Parse*
and strconv.Format*
for this linter (see e.g. plugins/inputs/dmcache/dmcache_linux.go
). In general I don't like to define constants for those per-se as often the meaning is clear at the usage place, e.g. defining default values for things in Init()
. Quite the contrary, often defining defaults as consts means you need to jump around in code to get the actual value...
So I'm inclined to not enable this one...
So I'm inclined to not enable this one...
Me too
Description
This issue starts a discussion about enabling:
Configuration:
maxLitCount
: (string) maximum number of instances of a string literal that are tolerated before warn.allowStrs
: (string) comma-separated list of allowed string literalsallowInts
: (string) comma-separated list of allowed integersallowFloats
: (string) comma-separated list of allowed floatsignoreFuncs
: (string) comma-separated list of function names regexp patterns to excludeExample configuration:
Expected output
Decision about enabling or not enabling this rule.
Findings
For this rule (with above configuration), the following findings were found in the current codebase: