Skip to content

Comments

Add is_empty methods to TxUpdate and {}_Response types#1961

Merged
evanlinjin merged 1 commit intobitcoindevkit:masterfrom
evanlinjin:feature/tx_update_is_empty
May 23, 2025
Merged

Add is_empty methods to TxUpdate and {}_Response types#1961
evanlinjin merged 1 commit intobitcoindevkit:masterfrom
evanlinjin:feature/tx_update_is_empty

Conversation

@evanlinjin
Copy link
Member

@evanlinjin evanlinjin commented May 18, 2025

Description

is_empty methods are helpful in various contexts so I added them.

Changelog notice

Added
  - `is_empty` methods to `TxUpdate`, `SyncResponse` and `FullScanResponse`

Checklists

All Submissions:

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

New Features:

  • I've added tests for the new feature
  • I've added docs for the new feature

@evanlinjin evanlinjin force-pushed the feature/tx_update_is_empty branch from 0cf6f5b to 6785145 Compare May 19, 2025 00:04
@evanlinjin evanlinjin marked this pull request as ready for review May 19, 2025 00:05
@evanlinjin evanlinjin self-assigned this May 19, 2025
@evanlinjin evanlinjin force-pushed the feature/tx_update_is_empty branch from 6785145 to 83e8a70 Compare May 19, 2025 11:59
Copy link
Contributor

@LagginTimes LagginTimes left a comment

Choose a reason for hiding this comment

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

Nit in doc comment. Otherwise, LGTM!

@notmandatory notmandatory added the new feature New feature or request label May 22, 2025
@notmandatory notmandatory moved this to Needs Review in BDK Chain May 22, 2025
@notmandatory notmandatory added this to the Wallet 2.0.0 milestone May 22, 2025
@LagginTimes LagginTimes force-pushed the feature/tx_update_is_empty branch from 83e8a70 to e9263f9 Compare May 23, 2025 10:20
Copy link
Contributor

@LagginTimes LagginTimes left a comment

Choose a reason for hiding this comment

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

ACK e9263f9

@evanlinjin evanlinjin merged commit ff7d703 into bitcoindevkit:master May 23, 2025
19 checks passed
@github-project-automation github-project-automation bot moved this from Needs Review to Done in BDK Chain May 23, 2025
@ValuedMammal ValuedMammal mentioned this pull request May 26, 2025
39 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new feature New feature or request

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants