Skip to content

Update repo and electrsd readme#618

Merged
tcharding merged 3 commits into
rust-bitcoin:masterfrom
jamillambert:0602-repo-readme
Jun 8, 2026
Merged

Update repo and electrsd readme#618
tcharding merged 3 commits into
rust-bitcoin:masterfrom
jamillambert:0602-repo-readme

Conversation

@jamillambert

@jamillambert jamillambert commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

In repo readme move bitcoind to the correct place alphabetically and add an entry for electrsd.

Update the links in the electrsd readme.

Order the list alphabetically.
@jamillambert jamillambert requested a review from tcharding as a code owner June 2, 2026 18:39
Comment thread README.md Outdated
- `bitreq/`: [`bitreq`](https://crates.io/crates/bitreq): Simple, minimal-dependency HTTP client, with optional features for proxies, async, and https.
- `client/`: [`corepc-client`](https://crates.io/crates/corepc-client): A blocking JSON-RPC client used to test `corepc-types`.
- `contrib/`: Contains scripts including one to run local regtest `bitcoind` nodes using versions specified in the config file. A template config file and bitcoind aliases are in the subdirectory `templates/`
- `electrsd/`: [`electrsd`](https://crates.io/crates/electrsd): Utility to run a regtest [electrs](https://github.com/romanz/electrs/) process connected to a given [bitcoind](https://github.com/RCasatta/bitcoind) instance.

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.

Do we mean to point the user to RCasatta/bitcoind?
Because https://github.com/RCasatta/bitcoind resolves to https://github.com/rust-bitcoin/bitcoind, which points back to corepc. (tangent but rust-bitcoin/bitcoind points to corepc node, should point to corepc bitcoind).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

It was a copy and paste from the electrsd readme. I have added a patch to update the link there and copied the changes to the repo readme.

The bitcoind link was stale. Update it and electrs to point to
crates.io.
@jamillambert jamillambert requested a review from RCasatta as a code owner June 3, 2026 09:14
@jamillambert jamillambert changed the title Update repo readme to include electrsd Update repo and electrsd readme Jun 3, 2026

@tcharding tcharding left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK f19ec0e

Comment thread electrsd/README.md
Comment on lines -77 to 78
* A free port is asked to the OS (a very low probability race condition is still possible)
* A free port is asked to the OS (a very low probability race condition is still possible)
* The process is killed when the struct goes out of scope no matter how the test finishes

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Unrelated whitespace change, merging anyways.

@tcharding tcharding merged commit aba0eba into rust-bitcoin:master Jun 8, 2026
67 of 68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants