Skip to content

more quic error log details #166

more quic error log details

more quic error log details #166

Triggered via push July 17, 2024 12:13
Status Success
Total duration 20s
Artifacts
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 11 warnings
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: common/custodian.go#L147
Function 'validateCustodianUpdateNodes' has too many statements (44 > 40) (funlen)
lint: common/decoding.go#L31
Function 'DecodeSnapshotWithTopo' has too many statements (47 > 40) (funlen)
lint: common/decoding.go#L102
Function 'DecodeTransaction' has too many statements (65 > 40) (funlen)
lint: common/decoding.go#L207
Function 'ReadInput' is too long (80 > 60) (funlen)
lint: common/decoding.go#L290
Function 'ReadOutput' is too long (63 > 60) (funlen)
lint: common/genesis.go#L42
Function 'BuildSnapshots' has too many statements (41 > 40) (funlen)
lint: common/node.go#L82
Function 'validateNodeCancel' has too many statements (62 > 40) (funlen)
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: common/transaction.go#L148
receiver-naming: receiver name signed should be consistent with previous receiver name tx for SignedTransaction (revive)
lint: common/transaction.go#L174
receiver-naming: receiver name signed should be consistent with previous receiver name tx for SignedTransaction (revive)
lint: common/transaction.go#L214
receiver-naming: receiver name signed should be consistent with previous receiver name tx for SignedTransaction (revive)
lint: common/custodian.go#L217
unused-parameter: parameter 'store' seems to be unused, consider removing or renaming it as _ (revive)
lint: common/deposit_test.go#L24
var-naming: var chainId should be chainID (revive)
lint: common/custodian_test.go#L163
unused-parameter: parameter 'require' seems to be unused, consider removing or renaming it as _ (revive)
lint: common/space.go#L8
var-naming: struct field NodeId should be NodeID (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/validation.go#L329
increment-decrement: should replace signers += 1 with signers++ (revive)