Skip to content

Expose ElectrumClient::ping method#689

Merged
thunderbiscuit merged 1 commit intobitcoindevkit:masterfrom
thunderbiscuit:feature/ping
Mar 10, 2025
Merged

Expose ElectrumClient::ping method#689
thunderbiscuit merged 1 commit intobitcoindevkit:masterfrom
thunderbiscuit:feature/ping

Conversation

@thunderbiscuit
Copy link
Copy Markdown
Member

@thunderbiscuit thunderbiscuit commented Mar 4, 2025

Somehow I had forgotten this one as part of #662.

The ping either returns the unit type or an error.

Changelog notice

Added:
    - New `ElectrumClient::ping` method [#689]

[#689]: https://github.com/bitcoindevkit/bdk-ffi/pull/689

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

Copy link
Copy Markdown
Collaborator

@reez reez left a comment

Choose a reason for hiding this comment

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

ACK 9b0e732

Looks pretty straightforward https://docs.rs/electrum-client/0.22.0/electrum_client/trait.ElectrumApi.html#tymethod.ping to me so this PR makes sense how it exposes that method

@thunderbiscuit thunderbiscuit merged commit 9b0e732 into bitcoindevkit:master Mar 10, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants