Skip to content

Add docs for EsploraAsyncExt and make doctests runnable#889

Merged
evanlinjin merged 1 commit intobitcoindevkit:masterfrom
evanlinjin:add-async-syncing-esplora
Mar 17, 2023
Merged

Add docs for EsploraAsyncExt and make doctests runnable#889
evanlinjin merged 1 commit intobitcoindevkit:masterfrom
evanlinjin:add-async-syncing-esplora

Conversation

@evanlinjin
Copy link
Copy Markdown
Member

Description

Add docs for EsploraAsyncExt and make various doctests runnable.

@evanlinjin evanlinjin added this to the 1.0.0-alpha.0 milestone Mar 15, 2023
@evanlinjin evanlinjin force-pushed the add-async-syncing-esplora branch 2 times, most recently from 261ca73 to 136a0b1 Compare March 15, 2023 05:09
Copy link
Copy Markdown
Contributor

@rajarshimaitra rajarshimaitra left a comment

Choose a reason for hiding this comment

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

tACK 136a0b1

Comment thread crates/esplora/src/async_ext.rs Outdated
@evanlinjin evanlinjin force-pushed the add-async-syncing-esplora branch from 136a0b1 to 2c4d9e1 Compare March 15, 2023 06:26
@evanlinjin evanlinjin force-pushed the add-async-syncing-esplora branch from 2c4d9e1 to 877b658 Compare March 16, 2023 05:08
@evanlinjin evanlinjin self-assigned this Mar 16, 2023
Copy link
Copy Markdown
Contributor

@rajarshimaitra rajarshimaitra left a comment

Choose a reason for hiding this comment

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

ACK 877b658

@evanlinjin evanlinjin merged commit 2ae69ca into bitcoindevkit:master Mar 17, 2023
@notmandatory notmandatory mentioned this pull request Mar 20, 2023
29 tasks
@ChidiChuks
Copy link
Copy Markdown

@evanlinjin Great work on this PR! Considering the increasing adoption of Miniscript for expressive and secure policy construction in Bitcoin, would it be worth referencing or aligning this implementation with ongoing Miniscript support? This could help ensure compatibility and future-proofing, especially for more complex script use cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants