Skip to content

Async client bitcoind network parsing#1426

Open
benalleng wants to merge 2 commits intopayjoin:masterfrom
benalleng:async-client-bitcoind-parsing
Open

Async client bitcoind network parsing#1426
benalleng wants to merge 2 commits intopayjoin:masterfrom
benalleng:async-client-bitcoind-parsing

Conversation

@benalleng
Copy link
Collaborator

Because rust-bitcoin and bitcoind have different naming schemes for the
network chains we cannot use a simple FromStr impl to parse them and
should instead use the built-in from_core_args method when referencing
bitcoind directly.

Pull Request Checklist

Please confirm the following before requesting review:

@benalleng benalleng changed the title Async client bitcoind parsing Async client bitcoind network parsing Mar 20, 2026
@benalleng benalleng force-pushed the async-client-bitcoind-parsing branch from bc6d10a to 3e3e489 Compare March 20, 2026 11:17
Because rust-bitcoin and bitcoind have different naming schemes for the
network chains we cannot use a simple FromStr impl to parse them and
should instead use the built-in from_core_args method when referencing
bitcoind directly.
@benalleng benalleng force-pushed the async-client-bitcoind-parsing branch from 3e3e489 to e169739 Compare March 20, 2026 11:22
@coveralls
Copy link
Collaborator

coveralls commented Mar 20, 2026

Pull Request Test Coverage Report for Build 23340659336

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 84.125%

Totals Coverage Status
Change from base Build 23300212261: 0.0%
Covered Lines: 10646
Relevant Lines: 12655

💛 - Coveralls

Copy link
Collaborator

@arminsabouri arminsabouri left a comment

Choose a reason for hiding this comment

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

utACK e169739

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.

3 participants