Skip to content

Conversation

@jamillambert
Copy link
Collaborator

@jamillambert jamillambert commented Sep 9, 2025

Why is it even there? it's not even an RPC, it's the transactions field of GetBlockTemplate which hasn't changed since v17.

Remove it.

Also remove an empty file left over from a previous change.

There was an empty file left over from a previous commit's code
reorginazation.

Remove the empty file.
Why is it even there? it's not even an RPC, it's the `transactions`
field of GetBlockTemplate which hasn't changed since v17.

Remove it.
Copy link
Member

@tcharding tcharding left a comment

Choose a reason for hiding this comment

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

ACK e85b1bf

@tcharding tcharding merged commit 2ddf47c into rust-bitcoin:master Sep 9, 2025
30 checks passed
blaze-smith470pm added a commit to blaze-smith470pm/corepc that referenced this pull request Sep 26, 2025
e85b1bf6d8521871d80dc5551d208b7d9f70f985 Remove BlockTemplateTransaction from v29 (Jamil Lambert, PhD)
da52d8f014503a072e9199d154caa87cdfdf1dd8 Remove empty file (Jamil Lambert, PhD)

Pull request description:

  Why is it even there? it's not even an RPC, it's the `transactions` field of `GetBlockTemplate` which hasn't changed since v17.

  Remove it.

  Also remove an empty file left over from a previous change.

ACKs for top commit:
  tcharding:
    ACK e85b1bf6d8521871d80dc5551d208b7d9f70f985

Tree-SHA512: dd574a99ca5488d75a42dc83920446188668a47d7ab88f830f8b655cbe190c70413558274a95c27d53e95057029990e7b1c1c89f40cb0f371c5bccabd3bb1efa
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants