Skip to content

WIP: Verify removal of Index trait bound #482

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

Closed
wants to merge 2 commits into from

Conversation

tcharding
Copy link
Member

This is not a merge candidate. The point of this PR is to verify rust-bitcoin/bitcoin_hashes#168

Verify we can still build after bitcoin_hashes removes the Index
trait bound and replaces it for AsRef.

Clippy default settings seemed to have changed introducing a few new
warnings.

warning: variable does not need to be mutable
warning: deref on an immutable reference
warning: returning the result of a `let` binding from a block

Fix them all in a single patch because CI has to pass for each patch.
Verify we can still build after `bitcoin_hashes` removes the `Index`
trait bound and replaces it for `AsRef`.
@tcharding tcharding force-pushed the dev-verfiy-rm-index branch from 4e24b73 to 5c56181 Compare August 11, 2022 22:23
@tcharding
Copy link
Member Author

Closing as bitcoin_hashes is going to be archived soon so the PR this verifies has been closed.

@tcharding tcharding closed this Nov 8, 2022
@tcharding tcharding deleted the dev-verfiy-rm-index branch October 9, 2023 22:10
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.

1 participant