Skip to content
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

Add '_ back into the BitStreamWriter #1601

Merged

Conversation

tcharding
Copy link
Member

Recently we merged commit 53d4fe66b57c255086def2b5e47afaddee776b75 to fix CI even though a better approach is to use '_ because it assists reading the code (shows that the bit stream writer is not writing from a reference since its writing a Copy type n).

Add back in the '_ (I forget what its called).

Recently we merged `commit 53d4fe6` to
fix CI even though a better approach is to use `'_` because it assists
reading the code (shows that the bit stream writer is not writing from a
reference since its writing a `Copy` type `n`).

Add back in the `'_` (I forget what its called).
@tcharding
Copy link
Member Author

Interesting, dtonlay musn't have updated the nightly toolchain yet since this is passing CI and not choking on the new rustfmt version like secp.

@apoelstra
Copy link
Member

apoelstra commented Jan 30, 2023

It looks like rust-bitcoin just doesn't have the required_version field. Never mind, it was just at the bottom of my screen.

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK 86f3727

Copy link
Collaborator

@Kixunil Kixunil left a comment

Choose a reason for hiding this comment

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

ACK 86f3727

@apoelstra apoelstra merged commit 6db85c1 into rust-bitcoin:master Feb 7, 2023
@tcharding tcharding deleted the 01-31-add-lifetime-back-in branch February 13, 2023 19:51
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.

None yet

3 participants