Skip to content

Commit

Permalink
.golangci.yml: remove mentions of fully deprecated linters
Browse files Browse the repository at this point in the history
  • Loading branch information
edigaryev committed Sep 25, 2024
1 parent 44e75d5 commit 4ea0275
Showing 1 changed file with 1 addition and 17 deletions.
18 changes: 1 addition & 17 deletions .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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

Expand Down Expand Up @@ -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
Expand Down

0 comments on commit 4ea0275

Please sign in to comment.