bump version to v0.18.11 #162
cedricfungAnnotations
10 errors and 11 warnings
|
lint:
crypto/blake2b_test.go#L58
Error return value of `x.Read` is not checked (errcheck)
|
|
lint:
p2p/sync.go#L102
Error return value of `me.SendSnapshotFinalizationMessage` is not checked (errcheck)
|
|
lint:
common/encoding.go#L314
Error return value of `enc.WriteByte` is not checked (errcheck)
|
|
lint:
common/encoding.go#L329
Error return value of `enc.WriteByte` is not checked (errcheck)
|
|
lint:
common/encoding.go#L341
Error return value of `enc.WriteByte` is not checked (errcheck)
|
|
lint:
kernel/boot.go#L17
Error return value of `node.loopCacheQueue` is not checked (errcheck)
|
|
lint:
kernel/cosi.go#L94
Error return value of `chain.node.Peer.SendTransactionRequestMessage` is not checked (errcheck)
|
|
lint:
kernel/cosi.go#L413
Error return value of `chain.AppendCosiAction` is not checked (errcheck)
|
|
lint:
kernel/cosi.go#L524
Error return value of `chain.AppendCosiAction` is not checked (errcheck)
|
|
lint:
kernel/cosi.go#L649
Error return value of `s.Signature.AggregateResponse` is not checked (errcheck)
|
|
lint
The following actions uses Node.js version which is deprecated and will be forced to run on node20: golangci/golangci-lint-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
lint:
util/base58/base58check_test.go#L61
redefines-builtin-id: redefinition of the built-in function len (revive)
|
|
lint:
config/reader.go#L14
var-naming: const KernelNetworkId should be KernelNetworkID (revive)
|
|
lint:
common/deposit_test.go#L24
var-naming: var chainId should be chainID (revive)
|
|
lint:
common/custodian_test.go#L14
var-naming: const mainnetId should be mainnetID (revive)
|
|
lint:
common/custodian_test.go#L128
var-naming: var nodeId should be nodeID (revive)
|
|
lint:
common/space.go#L8
var-naming: struct field NodeId should be NodeID (revive)
|
|
lint:
common/validation.go#L335
increment-decrement: should replace signers += 1 with signers++ (revive)
|
|
lint:
common/custodian_test.go#L163
unused-parameter: parameter 'require' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
common/node.go#L26
var-naming: method IdForNetwork should be IDForNetwork (revive)
|
|
lint:
common/asset.go#L11
var-naming: var XINAssetId should be XINAssetID (revive)
|