Skip to content

chain: add SubmitPackage API for atomic parents+child relay #783

chain: add SubmitPackage API for atomic parents+child relay

chain: add SubmitPackage API for atomic parents+child relay #783

Triggered via pull request May 5, 2025 16:58
Status Failure
Total duration 8m 9s
Artifacts

main.yml

on: pull_request
Format, compilation and lint check
1m 24s
Format, compilation and lint check
Matrix: run unit tests
Fit to window
Zoom out
Zoom in

Annotations

11 errors
run unit tests (unit-cover)
Process completed with exit code 2.
Format, compilation and lint check: chain/bitcoind_client.go#L1437
type assertion on error will fail on wrapped errors. Use errors.As to check for specific errors (errorlint)
Format, compilation and lint check: wallet/mock.go#L120
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"no child supplied\")" (err113)
Format, compilation and lint check: wallet/mock.go#L116
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"no parents supplied\")" (err113)
Format, compilation and lint check: chain/bitcoind_client.go#L1361
do not define dynamic errors, use wrapped static errors instead: "errors.New(\"submitpackage: child txn not defined\")" (err113)
Format, compilation and lint check: chain/bitcoind_client.go#L1356
do not define dynamic errors, use wrapped static errors instead: "errors.New(\"submitpackage: need at least one \" +\n\t\"parent txn\")" (err113)
Format, compilation and lint check: chain/bitcoind_client.go#L1364
Duplicate words (the) found (dupword)
Format, compilation and lint check: wallet/mock.go#L111
receiver-naming: receiver name c should be consistent with previous receiver name m for mockChainClient (revive)
Format, compilation and lint check: wallet/mock.go#L112
unused-parameter: parameter 'maxFeeRateBTCPerVByte' seems to be unused, consider removing or renaming it as _ (revive)
Format, compilation and lint check: chain/neutrino.go#L851
unused-parameter: parameter 'maxFeeRateBTCPerVByte' seems to be unused, consider removing or renaming it as _ (revive)
Format, compilation and lint check: chain/btcd.go#L646
unused-parameter: parameter 'maxFeeRateBTCPerVByte' seems to be unused, consider removing or renaming it as _ (revive)