Merge pull request #1021 from grishagavrin/master enable nilerr linter

Author: rekby

.golangci.yml

@@ -234,7 +234,6 @@ linters:
     - ireturn
     - maintidx
     - maligned
-    - nilerr
     - nlreturn
     - nonamedreturns
     - paralleltest

CHANGELOG.md

@@ -1,3 +1,6 @@
+* Refactored nil on err `internal/grpcwrapper/rawydb/issues.go`, when golangci-lint nilerr enabled
+* Refactored nil on err `internal/grpcwrapper/rawtopic/describe_topic.go`, when golangci-lint nilerr enabled
+
 ## v3.55.2
 * Fixed init info in topic writer, when autoseq num turned off.
 

internal/grpcwrapper/rawtopic/describe_topic.go

@@ -43,7 +43,7 @@ type DescribeTopicResult struct {
 
 func (res *DescribeTopicResult) FromProto(protoResponse *Ydb_Topic.DescribeTopicResponse) error {
 	if err := res.Operation.FromProtoWithStatusCheck(protoResponse.Operation); err != nil {
-		return nil
+		return err
 	}
 
 	protoResult := &Ydb_Topic.DescribeTopicResult{}

internal/grpcwrapper/rawydb/issues.go

@@ -17,7 +17,7 @@ func (issuesPointer *Issues) FromProto(p []*Ydb_Issue.IssueMessage) error {
 	issues := *issuesPointer
 	for i := range issues {
 		if err := issues[i].FromProto(p[i]); err != nil {
-			return nil
+			return err
 		}
 	}
 	return nil