Skip to content

Commit

Permalink
PMM-7 enable interfacebloat linter rule (#2316)
Browse files Browse the repository at this point in the history
* PMM-7 enable `interfacebloat` linter rule

* PMM-7 remove temp disables

---------

Co-authored-by: Artem Gavrilov <[email protected]>
  • Loading branch information
Alex Tymchuk and artemgavrilov authored Jun 28, 2023
1 parent 70f3748 commit fe8e947
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
1 change: 0 additions & 1 deletion .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,6 @@ linters:
- revive
- paralleltest
- gocognit
- interfacebloat
- errcheck
# ENDTODO

Expand Down
2 changes: 1 addition & 1 deletion admin/commands/pmm/server/docker/deps.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ import (
//go:generate ../../../../../bin/mockery -name=Functions -case=snake -inpkg -testonly

// Functions contain methods required to interact with Docker.
type Functions interface {
type Functions interface { //nolint:interfacebloat
Imager
Installer

Expand Down
2 changes: 1 addition & 1 deletion managed/services/management/dbaas/deps.go
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ type componentsService interface {
InstallOperator(context.Context, *dbaasv1beta1.InstallOperatorRequest) (*dbaasv1beta1.InstallOperatorResponse, error)
}

type kubernetesClient interface {
type kubernetesClient interface { //nolint:interfacebloat
SetKubeconfig(string) error
ListDatabaseClusters(context.Context) (*dbaasv1.DatabaseClusterList, error)
GetDatabaseCluster(context.Context, string) (*dbaasv1.DatabaseCluster, error)
Expand Down

0 comments on commit fe8e947

Please sign in to comment.