ArgsManager args and WalletContext context were previously used to create the wallet into FuzzedWallet. After fa15861763df71e788849b587883b3c16bb12229, they are not used anymore. This PR removes them.
fuzz: remove unused `args` and `context` from `FuzzedWallet` #29388
pull brunoerg wants to merge 1 commits into bitcoin:master from brunoerg:2024-02-fuzz-clean-fuzzedwallet changing 1 files +0 −3-
brunoerg commented at 8:33 PM on February 5, 2024: contributor
-
fuzz: remove unused `args` and `context` from `FuzzedWallet` b14298c5bc
-
DrahtBot commented at 8:33 PM on February 5, 2024: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage
For detailed information about the code coverage, see the test coverage report.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK maflcko, epiccurious, ryanofsky If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
- DrahtBot added the label Tests on Feb 5, 2024
-
- jo-elimu approved
-
maflcko commented at 7:38 AM on February 6, 2024: member
lgtm ACK b14298c5bca395e5ed6a27fe1758c0d1f4b824ac
-
epiccurious commented at 12:04 PM on February 6, 2024: contributor
utACK b14298c5bca395e5ed6a27fe1758c0d1f4b824ac
- ryanofsky approved
-
ryanofsky commented at 12:41 AM on February 7, 2024: contributor
Code review ACK b14298c5bca395e5ed6a27fe1758c0d1f4b824ac
- ryanofsky assigned ryanofsky on Feb 7, 2024
- ryanofsky merged this on Feb 7, 2024
- ryanofsky closed this on Feb 7, 2024
- bitcoin locked this on Feb 6, 2025
