Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

test(wallet): small cleanups to test_wallet_transactions_relevant #1785

Merged

Conversation

notmandatory
Copy link
Member

@notmandatory notmandatory commented Dec 20, 2024

Description

This cleans up the test_wallet_transactions_relevant test based on suggestions that didn't make it into #1779.

Notes to the reviewers

Also included a small use cleanup in wallet/mod.

Changelog notice

None.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

@luisschwab
Copy link
Contributor

ACK 6346536

Copy link
Contributor

@ValuedMammal ValuedMammal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 6346536

@ValuedMammal ValuedMammal merged commit a208144 into bitcoindevkit:master Dec 23, 2024
21 checks passed
@notmandatory notmandatory deleted the fix/wallet-transactions-test branch December 23, 2024 14:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants