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
internals: Release tracking PR v0.3.0
#2632
internals: Release tracking PR v0.3.0
#2632
Conversation
This will get red CI because of the current clippy problem. |
Pull Request Test Coverage Report for Build 8443857368Details
💛 - Coveralls |
02e5aa6
to
3d52557
Compare
3d52557
to
ff55c6d
Compare
CI is red because of our duplicate dependency lint. It is not red because of clippy, which shouldn't ever have a "current problem" because its version is pinned. |
Oh, you are correct. I thought there was a clippy warning at the moment in multiple places even with the pinning but looks like I was imagining things. All my complaining about slow dev process and this one was all me :) |
ff55c6d
to
33a0cd8
Compare
OMG few amusing fails yesterday, the "clippy problem" I had in mind is in the future #2629, goodness me. |
In 33a0cd8 I'm not exactly sure how CI is passing, but you definitely spelled |
In preparation for release add a changelog and bump the version number.
33a0cd8
to
af6dc1d
Compare
Because we are still not using |
I remember now, that particular pipe doesn't work with |
v0.3.0
@@ -105,7 +111,7 @@ version = "0.13.0" | |||
source = "registry+https://github.com/rust-lang/crates.io-index" | |||
checksum = "1930a4dabfebb8d7d9992db18ebe3ae2876f0a305fab206fd168df931ede293b" | |||
dependencies = [ | |||
"bitcoin-internals", | |||
"bitcoin-internals 0.2.0", |
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.
Is this supposed to 0.2.0?
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.
Yeah, until we release a new version of hashes
we will have two versions of internals
in play.
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.
Its when we release the new version of secp that the duplicate goes away (indirectly because of current secp
depending on hashes v0.13.0
depending on internals v0.2.0
).
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.
I miss-spoke, when we depend on the newly released version of secp.
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 af6dc1d
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 af6dc1d
Tag and publish please @apoelstra |
Then soon as its up we can release |
Oops. Tagged and published. |
In preparation for release add a changelog and bump the version number.
Please note, the changelog is pretty terse.