diff --git a/.golangci.yml b/.golangci.yml index 56bf400..0c27179 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -10,15 +10,6 @@ linters: enable-all: true disable: - # Messages like "struct of size 104 bytes could be of size 96 bytes" from a package - # that was last updated 2 years ago[1] are barely helpful. - # - # After all, we're writing the code for other people, so let's trust the compiler here (that's - # constantly evolving compared to this linter) and revisit this if memory usage becomes a problem. - # - # [1]: https://github.com/mdempsky/maligned/commit/6e39bd26a8c8b58c5a22129593044655a9e25959 - - maligned - # We don't have high-performance requirements at this moment, so sacrificing # the code readability for marginal performance gains is not worth it. - prealloc @@ -31,19 +22,12 @@ linters: # Unfortunately, we use globals due to how spf13/cobra works. - gochecknoglobals - # That's fine that some Proto objects don't have all fields initialized - - exhaustivestruct - # Style linters that are total nuts. - wsl - gofumpt - goimports - funlen - # This conflicts with the Protocol Buffers Version 3 design, - # which is largely based on default values for struct fields. - - exhaustivestruct - # Enough parallelism for now. - paralleltest @@ -82,7 +66,7 @@ linters: - exhaustruct # This is not a library, so it's OK to use dynamic errors - - goerr113 + - err113 # fmt.Sprintf() looks a bit nicer than string addition - perfsprint