Skip to content

Conversation

@fanquake
Copy link
Member

Rebase of #14649 (which itself was a rebase of #11256).

Squash the two test related commits, and swapped out size usage for vsize.

Added a commit with release notes.

@fanquake
Copy link
Member Author

@conscott @meshcollider @esotericnonsense did you want to re-review here? Maybe we'll get this in before the commits hit their second birthday 🍰 .

@cvengler
Copy link
Contributor

Concept ACK 17d178f

@esotericnonsense
Copy link
Contributor

utACK looks good to me (two years, ahh!)

Copy link
Contributor

@meshcollider meshcollider left a comment

Choose a reason for hiding this comment

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

utACK 17d178f

@instagibbs
Copy link
Member

utACK 17d178f

maflcko pushed a commit that referenced this pull request Aug 20, 2019
17d178f doc: add release-notes for getmempoolentry weight field addition (fanquake)
9c9cc2b qa: Add RPC tests for weight in mempool entry (Daniel Edgecumbe)
54aaa78 RPC: add weight to mempool entry output (Daniel Edgecumbe)

Pull request description:

  Rebase of #14649 (which itself was a rebase of #11256).

  Squash the two test related commits, and swapped out `size` usage for `vsize`.

  Added a commit with release notes.

ACKs for top commit:
  emilengler:
    Concept ACK 17d178f
  instagibbs:
    utACK 17d178f
  meshcollider:
    utACK 17d178f

Tree-SHA512: 1d354c9837e0ad0afa40325de9329b9e62688d5eab4d9e1cf9b46d8ae29d08f42d903ab37a41751c2ea8f9034231b21095881b1f5d911cb542b8b06bc85dc7cd
@maflcko maflcko merged commit 17d178f into bitcoin:master Aug 20, 2019
@fanquake fanquake deleted the rpc_mempoolentry_weight_rebased_release_notes branch August 20, 2019 21:48
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Aug 22, 2019
17d178f doc: add release-notes for getmempoolentry weight field addition (fanquake)
9c9cc2b qa: Add RPC tests for weight in mempool entry (Daniel Edgecumbe)
54aaa78 RPC: add weight to mempool entry output (Daniel Edgecumbe)

Pull request description:

  Rebase of bitcoin#14649 (which itself was a rebase of bitcoin#11256).

  Squash the two test related commits, and swapped out `size` usage for `vsize`.

  Added a commit with release notes.

ACKs for top commit:
  emilengler:
    Concept ACK 17d178f
  instagibbs:
    utACK bitcoin@17d178f
  meshcollider:
    utACK 17d178f

Tree-SHA512: 1d354c9837e0ad0afa40325de9329b9e62688d5eab4d9e1cf9b46d8ae29d08f42d903ab37a41751c2ea8f9034231b21095881b1f5d911cb542b8b06bc85dc7cd
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants