Skip to content

docs(rpc): add Emitter::next_block and Emitter::mempool examples#2210

Merged
ValuedMammal merged 1 commit into
bitcoindevkit:masterfrom
kwsantiago:docs/emitter-rustdoc-examples
Jul 11, 2026
Merged

docs(rpc): add Emitter::next_block and Emitter::mempool examples#2210
ValuedMammal merged 1 commit into
bitcoindevkit:masterfrom
kwsantiago:docs/emitter-rustdoc-examples

Conversation

@kwsantiago

@kwsantiago kwsantiago commented May 23, 2026

Copy link
Copy Markdown
Contributor

Adds # Example rustdoc to Emitter::next_block and Emitter::mempool, which were flagged as missing in the #2006 review.

Originally this PR also removed example_bitcoind_rpc_polling, but #2006 has since merged and removed all example crates, so that part is dropped after rebase. Only the doc additions remain.

Part of the #2006 follow-up to document the components previously covered by the examples/ crates.

@nymius nymius left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

code ACK d7e0362

Needs rebase.

@kwsantiago kwsantiago force-pushed the docs/emitter-rustdoc-examples branch from d7e0362 to 3c416e2 Compare May 25, 2026 13:58
@kwsantiago kwsantiago requested a review from nymius May 25, 2026 13:59
@codecov

codecov Bot commented May 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.32%. Comparing base (6d03fc3) to head (30fa179).

Files with missing lines Patch % Lines
crates/bitcoind_rpc/src/lib.rs 0.00% 24 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2210      +/-   ##
==========================================
- Coverage   78.65%   78.32%   -0.34%     
==========================================
  Files          30       30              
  Lines        5909     5934      +25     
  Branches      279      281       +2     
==========================================
  Hits         4648     4648              
- Misses       1185     1210      +25     
  Partials       76       76              
Flag Coverage Δ
rust 78.32% <0.00%> (-0.34%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ValuedMammal ValuedMammal added the documentation Improvements or additions to documentation label May 25, 2026
@nymius

nymius commented May 25, 2026

Copy link
Copy Markdown
Contributor

Could you rebase again?

@kwsantiago kwsantiago force-pushed the docs/emitter-rustdoc-examples branch from 3c416e2 to 78a1729 Compare May 26, 2026 01:20
@kwsantiago kwsantiago changed the title docs(rpc): add Emitter examples and remove example_bitcoind_rpc_polling docs(rpc): add Emitter::next_block and Emitter::mempool examples May 26, 2026

@nymius nymius left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ACK 78a1729

Comment thread crates/bitcoind_rpc/src/lib.rs Outdated
@kwsantiago kwsantiago force-pushed the docs/emitter-rustdoc-examples branch from 78a1729 to 0dfeb47 Compare June 7, 2026 11:38
Comment thread crates/bitcoind_rpc/src/lib.rs Outdated
@kwsantiago kwsantiago force-pushed the docs/emitter-rustdoc-examples branch from 0dfeb47 to 1f181c4 Compare June 9, 2026 21:55

@nymius nymius left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ACK 1f181c4

@ValuedMammal ValuedMammal force-pushed the docs/emitter-rustdoc-examples branch from 1f181c4 to 30fa179 Compare July 11, 2026 22:34

@ValuedMammal ValuedMammal 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 30fa179

@kwsantiago kwsantiago requested a review from evanlinjin July 11, 2026 22:43
@ValuedMammal ValuedMammal dismissed evanlinjin’s stale review July 11, 2026 22:50

Comments addressed

@ValuedMammal ValuedMammal merged commit fcfb7dd into bitcoindevkit:master Jul 11, 2026
13 of 15 checks passed
@kwsantiago kwsantiago deleted the docs/emitter-rustdoc-examples branch July 11, 2026 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants