go 1.23.2 has fixed the bugs #179
cedricfungAnnotations
10 errors and 11 warnings
|
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:
crypto/blake2b_test.go#L58
Error return value of `x.Read` 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 use a deprecated Node.js version 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:
crypto/rand.go#L32
increment-decrement: should replace set[b] += 1 with set[b]++ (revive)
|
|
lint:
crypto/blake2b_test.go#L18
unexported-return: exported func NewBlake2bXOF returns unexported type *crypto.xof, which can be annoying to use (revive)
|
|
lint:
crypto/rand.go#L10
unexported-return: exported func RandReader returns unexported type *crypto.randReader, which can be annoying to use (revive)
|
|
lint:
crypto/signature.go#L66
receiver-naming: receiver name publicKey should be consistent with previous receiver name privateKey for Key (revive)
|
|
lint:
crypto/signature.go#L81
receiver-naming: receiver name publicKey should be consistent with previous receiver name privateKey for Key (revive)
|
|
lint:
common/asset.go#L11
var-naming: var XINAssetId should be XINAssetID (revive)
|
|
lint:
common/custodian_test.go#L163
unused-parameter: parameter 'require' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
common/asset.go#L12
var-naming: var BitcoinAssetId should be BitcoinAssetID (revive)
|