`bitcoin_core_sv2` should leverage `impl Display` for printing messages

Open Beginner friendly
#404 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
72/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Quiet
Tech stack
rust
Domain
backend

Research direction

Read bitcoin-core-sv2/src/template_distribution_protocol/mod.rs at the referenced lines, then compare the analogous work in issue #403. Identify the existing message-printing paths and update the three referenced locations to use the intended Display implementation. Done means those messages consistently leverage Display without changing their meaning.

Written by the indexing model from the issue text.

Description

bitcoin-core-sv2 ux

similar to #403 but for bitcoin_core_sv2

https://github.com/stratum-mining/sv2-apps/blob/88bfb742d1824beb3d13fe875e1915a6bc7767d8/bitcoin-core-sv2/src/template_distribution_protocol/mod.rs#L207

https://github.com/stratum-mining/sv2-apps/blob/88bfb742d1824beb3d13fe875e1915a6bc7767d8/bitcoin-core-sv2/src/template_distribution_protocol/mod.rs#L210

https://github.com/stratum-mining/sv2-apps/blob/88bfb742d1824beb3d13fe875e1915a6bc7767d8/bitcoin-core-sv2/src/template_distribution_protocol/mod.rs#L235

Dominant language
Rust
Stars
18
Forks
40
Avg merge
1d 23h
Merged PRs (30d)
23

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from stratum-mining/sv2-apps

All issues in stratum-mining/sv2-apps

Similar issues

More Rust issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.