-
Notifications
You must be signed in to change notification settings - Fork 65
Fix latest clippy/rustc lints and release 0.11.1. #233
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
Fix latest clippy/rustc lints and release 0.11.1. #233
Conversation
There are only a few of these, and the lifetime ones are a compiler warning rather than a clippy warning (meaning that they will cause failures in my local CI) so fix them here.
c82cb05 to
527df49
Compare
clarkmoody
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK 527df49
Looks like CI is still failing for some reason.
|
I'll just rerun the API checker and commit it. In rust-bitcoin we've pinned the version of cargo-public-api to avoid these sorts of problems but here it's probably okay to just leave it, at least for this old version. |
The output of cargo-public-api changed at some point.
527df49 to
d477dec
Compare
tcharding
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK d477dec
|
cc @clarkmoody can you re-ACK? |
clarkmoody
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK d477dec
|
Tagged and published. |
The latest versions of rust-bitcoin and rust-elements depend on bech32 0.11.0. Unfortunately this has been hit by a recent breaking change in rustc rust-lang/rust#138907
Work around the break and cut a new version. While we're at it, fix all the recent lint changes (there are not many).