Skip to content

Conversation

@t-bast
Copy link
Collaborator

@t-bast t-bast commented May 21, 2024

This is a follow-up to #1092 that fixes the following issues:

  • fix a few typos
  • remove non-zero-fee anchors test cases
  • remove remote_pubkey rotation

This is a follow-up to lightning#1092
that fixes the following issues:

- fix a few typos
- remove non-zero-fee anchors test cases
- remove `remote_pubkey` rotation
@t-bast t-bast requested a review from rustyrussell May 21, 2024 08:59
Copy link
Contributor

@vincenzopalazzo vincenzopalazzo left a comment

Choose a reason for hiding this comment

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

ACK 3e9b572

Thanks, and sorry if we missed some of the follow-up comments.

Copy link
Collaborator

@Roasbeef Roasbeef left a comment

Choose a reason for hiding this comment

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

LGTM 🐘

## Key Derivation

Each commitment transaction uses a unique `localpubkey`, and a `remotepubkey`.
Each commitment transaction uses a unique `localpubkey`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

@Roasbeef Roasbeef merged commit 57ce4b1 into lightning:master Jun 3, 2024
@t-bast t-bast deleted the remove-old-features-typos branch June 3, 2024 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants