Skip to content

Cut 0.0.100 #1044

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Aug 17, 2021
Merged

Cut 0.0.100 #1044

merged 2 commits into from
Aug 17, 2021

Conversation

TheBlueMatt
Copy link
Collaborator

Dependent on #1011, this cuts 0.0.100

@TheBlueMatt TheBlueMatt added this to the 0.0.100 milestone Aug 14, 2021
@codecov
Copy link

codecov bot commented Aug 14, 2021

Codecov Report

Merging #1044 (e619b23) into main (6f16453) will decrease coverage by 0.01%.
The diff coverage is n/a.

❗ Current head e619b23 differs from pull request most recent head 11f7879. Consider uploading reports for the commit 11f7879 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1044      +/-   ##
==========================================
- Coverage   90.84%   90.82%   -0.02%     
==========================================
  Files          65       65              
  Lines       32795    32795              
==========================================
- Hits        29792    29786       -6     
- Misses       3003     3009       +6     
Impacted Files Coverage Δ
lightning/src/ln/functional_tests.rs 97.40% <0.00%> (-0.10%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6f16453...11f7879. Read the comment docs.

@jkczyz jkczyz self-requested a review August 17, 2021 00:38
Copy link
Contributor

@valentinewallace valentinewallace left a comment

Choose a reason for hiding this comment

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

ACK mod comments

CHANGELOG.md Outdated
Comment on lines 64 to 68
* If
`lightning::chain::keysinterface::KeysInterface::get_shutdown_scriptpubkey`
returns a `ShutdownScript` which was constructed using a constructor other
than `ShutdownScript::new_p2wpkh_from_pubkey`, the `ChannelManager` will not
be readable by versions prior to 0.0.100. Further, versions prior to 0.0.100
Copy link
Contributor

Choose a reason for hiding this comment

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

That method is pub(crate) so you'll never see a ShutdownScript constructed that way in user code. So any new channels created will not be readable. However, any channel whose shutdown script was created using the old get_shutdown_pubkey API will continue to be readable.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hmm, should we make that method pub?

Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure. Actually, KeysManager is currently usingnew_p2wpkh_from_pubkey, so anyone using it for their KeysInterface shouldn't run into this problem.

@TheBlueMatt
Copy link
Collaborator Author

Tweaked changelog wordings to address feedback.

@TheBlueMatt
Copy link
Collaborator Author

Squashed without changes: 0b62f16b...11f78798

@TheBlueMatt TheBlueMatt merged commit aacb7fb into lightningdevkit:main Aug 17, 2021
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.

3 participants