FundTx
was used by the wallet_notifications
target which we recently removed. So it’s now unused and can be removed.
FundTx
from FuzzedWallet
#32772
FundTx
was used by the wallet_notifications
target which we recently removed. So it’s now unused and can be removed.
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32772.
See the guideline for information on the review process.
Type | Reviewers |
---|---|
ACK | maflcko, dergoegge, kevkevinpal |
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
Reviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
ACK cd1ae1b
makes sense to remove dead code