diff options
author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-08-10 13:38:49 +0300 |
---|---|---|
committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-08-11 15:03:14 +0300 |
commit | 7e87131edb431c2d7d491f29606fe88955b399eb (patch) | |
tree | 104e953f6c154c9f4b656008103ba19f6b9d768a /client | |
parent | f5d1a379260156181b599d1b45e162212a67e399 (diff) |
golangci-lint: Allow `testing.T` as first parameter
We currently exclude a revive rule that `context.Context` should be the
first parameter for our test sources. This can be handled better though
because golangci-lint allows us to exclude certain types from this rule.
Adapt the rule to allow `testing.T` et al before `context.Context` and
remove the excluded rule. Interestingly, this now surfaces a whole bunch
of `nolint: revive` annotations that aren't needed anymore, so we fix
them in the same commit.
Diffstat (limited to 'client')
-rw-r--r-- | client/pool_options.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/client/pool_options.go b/client/pool_options.go index 35f924c41..decb2a812 100644 --- a/client/pool_options.go +++ b/client/pool_options.go @@ -7,7 +7,7 @@ type poolOptions struct { dialOptions []grpc.DialOption } -//nolint: revive,stylecheck // This is unintentionally missing documentation. +//nolint: stylecheck // This is unintentionally missing documentation. type PoolOption func(*poolOptions) func applyPoolOptions(options []PoolOption) *poolOptions { |