diff options
| author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-03-15 12:39:08 +0300 |
|---|---|---|
| committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-03-24 16:52:49 +0300 |
| commit | de764b05589ec73e1aa9ddd2d7118ced301602f0 (patch) | |
| tree | 1a2b4a22c0bb7f4daeb900df448a843f0fbf69e9 /.golangci.yml | |
| parent | 7b5acbe6ed71a017d9075df9b25ab58044476b98 (diff) | |
nodes: Silence warnings about ignored errors for `checkNodes()`
Both the local and SQL electors don't check errors returned by
`checkNodes()`. While we could convert all sites to properly do this, it
doesn't really feel worth it: both electors only exist due to legacy
reasons and should not be used in production systems at all.
Ignore most of those errors inline so that we can remove the excemptions
from golangci-lint's config file.
Diffstat (limited to '.golangci.yml')
| -rw-r--r-- | .golangci.yml | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/.golangci.yml b/.golangci.yml index 226847520..417897f8c 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -79,18 +79,6 @@ issues: text: "Error return value of `[^`]+.(Close|Serve)` is not checked" - linters: - errcheck - path: "internal/praefect/nodes/local_elector.go" - text: "Error return value of `s.checkNodes` is not checked" - - linters: - - errcheck - path: "internal/praefect/nodes/manager.go" - text: "Error return value of `strategy.checkNodes` is not checked" - - linters: - - errcheck - path: "internal/praefect/nodes/sql_elector.go" - text: "Error return value of `s.checkNodes` is not checked" - - linters: - - errcheck path: "internal/middleware/limithandler/limithandler.go" text: "Error return value of `limiter.Limit` is not checked" - linters: |