Skip to content

Conversation

@edilmedeiros
Copy link
Contributor

Commit 6092f89 renamed the bitcoind directory to be node and the BitcoinD type to Node, but the examples in the readme did not reflect the changes.

Commit 6092f89 renamed the `bitcoind` directory
to be `node` and the `BitcoinD` type to `Node`, but the examples in the readme
did not reflect the changes.
Copy link
Member

@tcharding tcharding left a comment

Choose a reason for hiding this comment

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

ACK 6268b6b

@tcharding
Copy link
Member

Legend! Thanks for the PR.


Startup options could be configured via the [`Conf`] struct using [`BitcoinD::with_conf`] or
[`BitcoinD::from_downloaded_with_conf`]
Startup options could be configured via the [`Conf`] struct using [`Node::with_conf`] or
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Startup options could be configured via the [`Conf`] struct using [`Node::with_conf`] or
Startup options could be configured via the [`Conf`] struct using [`Node::with_conf`] or

@tcharding tcharding merged commit 69461aa into rust-bitcoin:master Dec 19, 2024
27 checks passed
@tcharding
Copy link
Member

Woops, I got trigger happy and merged while the trailing whitespace was there - my bad.

@tcharding
Copy link
Member

Fixed in #39.

blaze-smith470pm added a commit to blaze-smith470pm/corepc that referenced this pull request Sep 26, 2025
6268b6becda1d95d55b57fea0536d6c5f809db39 Update readme (Edil Medeiros)

Pull request description:

  Commit 6092f8944cd565d3ee5523e7e7d2910021d49979 renamed the `bitcoind` directory to be `node` and the `BitcoinD` type to `Node`, but the examples in the readme did not reflect the changes.

ACKs for top commit:
  tcharding:
    ACK 6268b6becda1d95d55b57fea0536d6c5f809db39

Tree-SHA512: 311f42708a5a87136e77f5f01b03db5010ca9cd2ccb5ed0517986847037d320f0236a6ea1698f85d63df81ee49856eb9ac67c35dbf3e8badf5965edceb123de2
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