Skip to content

Add TxGraph::get_last_evicted#1977

Merged
evanlinjin merged 1 commit intobitcoindevkit:masterfrom
evanlinjin:feature/tx-graph-get-last-evicted
Jul 18, 2025
Merged

Add TxGraph::get_last_evicted#1977
evanlinjin merged 1 commit intobitcoindevkit:masterfrom
evanlinjin:feature/tx-graph-get-last-evicted

Conversation

@evanlinjin
Copy link
Copy Markdown
Member

Description

Ideally, this would be included as a field in TxNode, however that would be a breaking change.

Changelog notice

Added
- `TxGraph::get_last_evicted`

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo +nightly 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 self-assigned this Jun 14, 2025
@evanlinjin evanlinjin moved this to Needs Review in BDK Chain Jun 14, 2025
Copy link
Copy Markdown
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 a8bad67

Copy link
Copy Markdown
Collaborator

@oleonardolima oleonardolima left a comment

Choose a reason for hiding this comment

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

ACK a8bad67

@oleonardolima
Copy link
Copy Markdown
Collaborator

It just needs a rebase to run with the latest CI changes.

@evanlinjin evanlinjin force-pushed the feature/tx-graph-get-last-evicted branch from a8bad67 to 5d7da73 Compare July 10, 2025 06:33
Ideally, this would be included as a field in `TxNode`, however that
would be a breaking change.
@evanlinjin evanlinjin force-pushed the feature/tx-graph-get-last-evicted branch from 5d7da73 to b555acb Compare July 18, 2025 11:22
Copy link
Copy Markdown
Member Author

@evanlinjin evanlinjin left a comment

Choose a reason for hiding this comment

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

self-ACK b555acb

@evanlinjin evanlinjin merged commit 23dfd6e into bitcoindevkit:master Jul 18, 2025
19 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Review to Done in BDK Chain Jul 18, 2025
@oleonardolima oleonardolima mentioned this pull request Jul 31, 2025
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants