Small fixes #607

Merged
fyrchik merged 4 commits from fyrchik/frostfs-node:gopatch-example into master 2023-08-17 16:50:04 +00:00

4 commits

Author SHA1 Message Date
18082ab608 [#607] .golangci.yml: Fix pre-commit warnings
Some checks failed
Tests and linters / Tests (1.21) (pull_request) Failing after 1m16s
DCO action / DCO (pull_request) Successful in 3m31s
Vulncheck / Vulncheck (pull_request) Successful in 3m42s
Build / Build Components (1.21) (pull_request) Successful in 4m48s
Tests and linters / Staticcheck (pull_request) Successful in 5m24s
Build / Build Components (1.20) (pull_request) Successful in 6m59s
Tests and linters / Tests with -race (pull_request) Failing after 9m47s
Tests and linters / Lint (pull_request) Successful in 10m22s
Tests and linters / Tests (1.20) (pull_request) Successful in 10m31s
Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
2023-08-14 14:25:55 +03:00
6d10e07565 [#607] *: Use keys.PublicKeys.Copy() where possible
Some checks failed
Build / Build Components (1.21) (pull_request) Successful in 8m20s
DCO action / DCO (pull_request) Successful in 8m51s
Vulncheck / Vulncheck (pull_request) Successful in 10m9s
Build / Build Components (1.20) (pull_request) Successful in 11m29s
Tests and linters / Staticcheck (pull_request) Successful in 16m5s
Tests and linters / Tests (1.21) (pull_request) Failing after 16m26s
Tests and linters / Tests (1.20) (pull_request) Failing after 16m34s
Tests and linters / Lint (pull_request) Successful in 19m40s
Tests and linters / Tests with -race (pull_request) Failing after 19m40s
Semantic patch:
```
@@
var dst identifier
var src identifier
var keys identifier
@@
 import keys "github.com/nspcc-dev/neo-go/pkg/crypto/keys"

-dst := make(keys.PublicKeys, len(src))
-copy(dst, src)
+dst := src.Copy()
```

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
2023-08-14 14:24:52 +03:00
6669494af4 [#607] *: Use zap.Stringer() where possible
Semantic patch:
```
@@
var f expression
var t expression
var a expression
@@
 f(
    ...,
-    zap.String(t, a.String()),
+    zap.Stringer(t, a),
    ...,
)
```

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
2023-08-14 14:21:24 +03:00
3a96a7f2c7 [#607] *: Remove redundant if on error returns
Semantic patch:
```
@@
@@
-if err != nil { return err }
-return nil
+return err
```

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
2023-08-14 14:21:24 +03:00