Enabled structcheck linter #1219
Conversation
| @@ -19,8 +19,6 @@ import ( | |||
| ) | |||
|
|
|||
| type testItem struct { | |||
| v uint32 | |||
There was a problem hiding this comment.
This line was removed because the structcheck linter wrote internal/pool/pool_test.go:22:2: "v" is unused (structcheck) v uint32
| @@ -23,7 +23,6 @@ type PublicMessage struct { | |||
| // partitioning at level message available by protocol, but doesn't available by current server implementation | |||
| // the field hidden from public access for prevent runtime errors. | |||
| // it will be published after implementation on server side. | |||
| futurePartitioning PublicFuturePartitioning | |||
There was a problem hiding this comment.
This line was removed because the structcheck linter wrote internal/topic/topicwriterinternal/message.go:26:2: "futurePartitioning" is unused (structcheck) futurePartitioning PublicFuturePartitioning
| @@ -22,5 +21,4 @@ type WritersCommonConfig struct { | |||
| cred credentials.Credentials | |||
| credUpdateInterval time.Duration | |||
| clock clockwork.Clock | |||
| forceCodec rawtopiccommon.Codec | |||
There was a problem hiding this comment.
This line was removed because the structcheck linter wrote internal/topic/topicwriterinternal/writer_config.go:25:2: "forceCodec" is unused (structcheck) forceCodec rawtopiccommon.Codec
| @@ -6,7 +6,6 @@ import ( | |||
| "github.com/jonboulle/clockwork" | |||
|
|
|||
| "github.com/ydb-platform/ydb-go-sdk/v3/credentials" | |||
| "github.com/ydb-platform/ydb-go-sdk/v3/internal/grpcwrapper/rawtopic/rawtopiccommon" | |||
There was a problem hiding this comment.
I didn’t remove this, but when I added it back, the development environment began to issue “unused import”.
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
Pull request type
Please check the type of change your PR introduces:
What is the current behavior?
Issue Number: #966
What is the new behavior?
Other information