Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Nov 29, 2022

Currently devs are free to set or not set the wallet type in the test_runner when only one type is allowed to be set.

This is inconsistent and causes review comments such as:

@DrahtBot
Copy link
Contributor

DrahtBot commented Nov 29, 2022

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK achow101

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #26152 (Bump unconfirmed ancestor transactions to target feerate by Xekyo)
  • #20892 (tests: Run both descriptor and legacy tests within a single test invocation by achow101)

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.

@achow101
Copy link
Member

ACK fadb869

@maflcko maflcko merged commit 3be2106 into bitcoin:master Nov 30, 2022
@maflcko maflcko deleted the 2211-test-wallet-type-💤 branch November 30, 2022 09:30
@hebasto hebasto modified the milestone: 24.0.1 Nov 30, 2022
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Dec 1, 2022
@bitcoin bitcoin locked and limited conversation to collaborators Nov 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants