Skip to content

Conversation

@dzutto
Copy link

@dzutto dzutto commented Oct 6, 2021

No description provided.

PastaPastaPasta
PastaPastaPasta previously approved these changes Oct 9, 2021
Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

utACK for squash merge

I didn't review or correctness, just for saftey, as in this PR only does what it says it does, and nothing more.

@UdjinM6 UdjinM6 added this to the 18 milestone Oct 10, 2021
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

Few more tiny issues, see below

laanwj and others added 2 commits October 11, 2021 12:22
fa5e045 rpc: Documentation fixups (MarcoFalke)
fa91e8e Use RPCHelpMan for all RPCs (MarcoFalke)
fa520e7 lint: Must use RPCHelpMan to generate the RPC docs (MarcoFalke)

Pull request description:

  The resulting documentation should not change unless the type in the oneline-summary was previously incorrect. (E.g. string vs bool)

Tree-SHA512: 4ff355b6a53178f02781e97a7aca7ee1d0d97ff348b6bf5a01caa1c96904ee33c704465fae54c2cd7445097427fd04c71ad3779bb7a7ed886055ef36c1b5a1d0
UdjinM6
UdjinM6 previously approved these changes Oct 11, 2021
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

LGTM, utACK

Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

utACK for squash merge

@UdjinM6 UdjinM6 dismissed their stale review October 11, 2021 21:33

ci isn't happy

Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

re-utACK

@UdjinM6 UdjinM6 merged commit 0f2e8aa into dashpay:develop Oct 11, 2021
@dzutto dzutto deleted the merge_14726 branch October 11, 2021 21:59
pravblockc pushed a commit to pravblockc/dash that referenced this pull request Nov 18, 2021
* Merge bitcoin#14726: Use RPCHelpMan for all RPCs

fa5e045 rpc: Documentation fixups (MarcoFalke)
fa91e8e Use RPCHelpMan for all RPCs (MarcoFalke)
fa520e7 lint: Must use RPCHelpMan to generate the RPC docs (MarcoFalke)

Pull request description:

  The resulting documentation should not change unless the type in the oneline-summary was previously incorrect. (E.g. string vs bool)

Tree-SHA512: 4ff355b6a53178f02781e97a7aca7ee1d0d97ff348b6bf5a01caa1c96904ee33c704465fae54c2cd7445097427fd04c71ad3779bb7a7ed886055ef36c1b5a1d0

* Dash-specific changes to support RPCHelpMan with RPC commands

Signed-off-by: Dzutte <[email protected]>

Co-authored-by: Wladimir J. van der Laan <[email protected]>
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Jun 17, 2023
* Merge bitcoin#14726: Use RPCHelpMan for all RPCs

fa5e045 rpc: Documentation fixups (MarcoFalke)
fa91e8e Use RPCHelpMan for all RPCs (MarcoFalke)
fa520e7 lint: Must use RPCHelpMan to generate the RPC docs (MarcoFalke)

Pull request description:

  The resulting documentation should not change unless the type in the oneline-summary was previously incorrect. (E.g. string vs bool)

Tree-SHA512: 4ff355b6a53178f02781e97a7aca7ee1d0d97ff348b6bf5a01caa1c96904ee33c704465fae54c2cd7445097427fd04c71ad3779bb7a7ed886055ef36c1b5a1d0

* Dash-specific changes to support RPCHelpMan with RPC commands

Signed-off-by: Dzutte <[email protected]>

Co-authored-by: Wladimir J. van der Laan <[email protected]>
gades pushed a commit to piratecash/pirate that referenced this pull request Dec 9, 2023
* Merge bitcoin#14726: Use RPCHelpMan for all RPCs

fa5e045 rpc: Documentation fixups (MarcoFalke)
fa91e8e Use RPCHelpMan for all RPCs (MarcoFalke)
fa520e7 lint: Must use RPCHelpMan to generate the RPC docs (MarcoFalke)

Pull request description:

  The resulting documentation should not change unless the type in the oneline-summary was previously incorrect. (E.g. string vs bool)

Tree-SHA512: 4ff355b6a53178f02781e97a7aca7ee1d0d97ff348b6bf5a01caa1c96904ee33c704465fae54c2cd7445097427fd04c71ad3779bb7a7ed886055ef36c1b5a1d0

* Dash-specific changes to support RPCHelpMan with RPC commands

Signed-off-by: Dzutte <[email protected]>

Co-authored-by: Wladimir J. van der Laan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants