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 additional changelog entry #557

Merged
merged 2 commits into from Dec 9, 2022
Merged

Conversation

tcharding
Copy link
Member

@tcharding tcharding commented Dec 5, 2022

Bump version to 0.25.1 ready to release Add changelog entry for the recently fixed unsoundness issue.

Patch 1 is an annoyingly trivial fix to docs.

@apoelstra
Copy link
Member

We never released 0.25.0 so we can just stick with that version number.

@Kixunil
Copy link
Collaborator

Kixunil commented Dec 7, 2022

Awesome, so I didn't have to change the report. :D

@apoelstra
Copy link
Member

@tcharding can you update this PR so it updates the changelog and does nothing else?

@tcharding
Copy link
Member Author

Done, thanks for catching my mistake.

@tcharding tcharding changed the title Bump 0.25.1 Add additional changelog entry Dec 7, 2022
@tcharding
Copy link
Member Author

and does nothing else?

I hope patch 1 is ok, its in there so the backports and this all have exactly the same rustdoc code.

Kixunil
Kixunil previously approved these changes Dec 7, 2022
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 a929f28

(Ignoring that the commit says adding line before heading while it should be after.)

As is customary add a newline between rustdoc heading and content. Done
so that the code is identical to other released code (during backport
the space was added).
@tcharding
Copy link
Member Author

I fixed the commit message since this repo does not require two acks @apoelstra will be able to merge it still with his ack. Thanks

@apoelstra
Copy link
Member

@tcharding this commit adds trailing whitespace at the top of the changelog file.

@apoelstra
Copy link
Member

Also sorry, we have to fix #562 before release.

Recently we found and fixed an unsoundness issue in the
`preallocated_gen_new` function. As we have yet to release 0.25.0 we can
just update the changelog to reflect the newly merged fix.
@tcharding
Copy link
Member Author

tcharding commented Dec 8, 2022

@tcharding this commit adds trailing whitespace at the top of the changelog file.

Strange, I wonder why the githook didn't catch this? Even more strange I wonder why I push changes still to this day without reading the diff ... I definitely owe you a beer for this one.

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 494b07a

@apoelstra apoelstra merged commit ff4be18 into rust-bitcoin:master Dec 9, 2022
@apoelstra
Copy link
Member

Lol, jeez. Since #551 we need to do a major rev of secp256k1-sys, and then we can release 0.25.0.

@apoelstra
Copy link
Member

Ah, #549 does this. But it needs a rebase.

@tcharding tcharding deleted the bump-0.25.1 branch December 11, 2022 22:12
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