chain: add SubmitPackage API for atomic parents+child relay #783
main.yml
on: pull_request
Format, compilation and lint check
1m 24s
Matrix: run unit tests
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)
|