Skip to content

Conversation

theStack
Copy link
Contributor

This is a tiny documentation follow-up to #23549.

@DrahtBot DrahtBot added the Docs label Oct 20, 2022
@theStack theStack force-pushed the 202210-doc-add_scanblocks_to_doc_descriptors branch from 9397853 to ff138f9 Compare October 20, 2022 21:25
@theStack
Copy link
Contributor Author

@kouloumos @aureleoules: Thanks for your quick reviews! :D Took kind of a combination of both of your suggestions.

Copy link
Contributor

@aureleoules aureleoules left a comment

Choose a reason for hiding this comment

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

ACK ff138f9

Copy link
Contributor

@shaavan shaavan left a comment

Choose a reason for hiding this comment

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

ACK ff138f9

Clean documentation. Clearly explains what scanblocks does.

@maflcko maflcko merged commit 6d40484 into bitcoin:master Oct 21, 2022
@theStack theStack deleted the 202210-doc-add_scanblocks_to_doc_descriptors branch October 21, 2022 09:35
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Oct 23, 2022
ff138f9 doc: add `scanblocks` to list of descriptor RPCs (Sebastian Falbesoner)

Pull request description:

  This is a tiny documentation follow-up to bitcoin#23549.

ACKs for top commit:
  aureleoules:
    ACK ff138f9
  shaavan:
    ACK ff138f9

Tree-SHA512: cc45b78c13ec4aa5bac688648f8e83f04a9ae54376e67371b280428f0253e2585cf0575fa51e3835d4c13c940559bfcdd88d7504bf97a81b2a73bb34a0db7577
@bitcoin bitcoin locked and limited conversation to collaborators Oct 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants