Skip to content

Remove deprecated BlockSummary and get_blocks#225

Merged
luisschwab merged 2 commits into
bitcoindevkit:masterfrom
luisschwab:chore/remove-block-summary
Jul 8, 2026
Merged

Remove deprecated BlockSummary and get_blocks#225
luisschwab merged 2 commits into
bitcoindevkit:masterfrom
luisschwab:chore/remove-block-summary

Conversation

@luisschwab

@luisschwab luisschwab commented Jul 3, 2026

Copy link
Copy Markdown
Member

Closes #146

Changelog

- Remove deprecated `BlockSummary` type and `get_block` client method
- Bump stable toolchain to v1.96.1 and nightly to nightly-2026-07-03

@luisschwab luisschwab self-assigned this Jul 3, 2026
@luisschwab luisschwab requested a review from oleonardolima as a code owner July 3, 2026 18:57
@luisschwab

Copy link
Copy Markdown
Member Author

ping @oleonardolima

@oleonardolima oleonardolima left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

cACK d2d3918

You need to update the commit message, as it's a breaking change.

@oleonardolima oleonardolima added the api A breaking API change label Jul 8, 2026
@oleonardolima oleonardolima moved this to In Progress in BDK Chain Jul 8, 2026
@luisschwab luisschwab force-pushed the chore/remove-block-summary branch from d2d3918 to 32f76da Compare July 8, 2026 14:14
@luisschwab

Copy link
Copy Markdown
Member Author

CI has been super flaky lately...

@luisschwab luisschwab requested a review from oleonardolima July 8, 2026 14:34

@oleonardolima oleonardolima left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ACK 32f76da

@luisschwab luisschwab merged commit 531deaf into bitcoindevkit:master Jul 8, 2026
21 of 24 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in BDK Chain Jul 8, 2026
@luisschwab luisschwab deleted the chore/remove-block-summary branch July 8, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api A breaking API change

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Remove BlockSummary in favor of BlockInfo

2 participants