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

release: v0.29.0 #1267

Merged
merged 6 commits into from
Feb 17, 2023
Merged

release: v0.29.0 #1267

merged 6 commits into from
Feb 17, 2023

Conversation

erwanor
Copy link
Collaborator

@erwanor erwanor commented Feb 17, 2023

Hey there, this fixes #1263 :)

  • Referenced an issue explaining the need for the change
  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Added entry in .changelog/

@erwanor erwanor changed the title changelog: add v0.29.0 release: v0.29.0 Feb 17, 2023
@erwanor erwanor marked this pull request as ready for review February 17, 2023 19:40
Copy link
Contributor

@thanethomson thanethomson left a comment

Choose a reason for hiding this comment

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

Thanks very much for this @erwanor!

There are a couple of places on this branch where the html_root_url version hasn't been updated to v0.29.0:

  • tendermint/src/lib.rs
  • light-client/src/lib.rs
  • p2p/src/lib.rs

Also, note that pushing a v0.29.0 tag will not yet automatically build and push the crates to crates.io - this is still quite a manual process that involves running the release.sh script from the root of the repo.

We still need a GitHub workflow to do this (please speak to @hdevalence about this, as I shared details a few days ago about what we need there).

@erwanor
Copy link
Collaborator Author

erwanor commented Feb 17, 2023

@thanethomson Thanks for catching those, I'll update the docs!

@thanethomson
Copy link
Contributor

Ah, and I just noticed you're missing the .changelog/v0.29.0 folder.

@erwanor
Copy link
Collaborator Author

erwanor commented Feb 17, 2023

@thanethomson re: release.sh do I have the appropriate permissions to push to crates.io once this PR gets approved? or is this something you would need to run yourself? I can sync with henry about how we can help there, but I'm wondering if there's a ticket or convo that scopes the need (is there a blocker?)

@tarcieri
Copy link
Contributor

Note: if you’re using docs.rs, setting html_root_url manually is obsolete: rust-lang/api-guidelines#229

@codecov-commenter
Copy link

codecov-commenter commented Feb 17, 2023

Codecov Report

Merging #1267 (1c29869) into main (89c8add) will decrease coverage by 0.1%.
The diff coverage is n/a.

❗ Current head 1c29869 differs from pull request most recent head a65df6b. Consider uploading reports for the commit a65df6b to get more accurate results

@@           Coverage Diff           @@
##            main   #1267     +/-   ##
=======================================
- Coverage   64.3%   64.3%   -0.1%     
=======================================
  Files        250     250             
  Lines      21618   21618             
=======================================
- Hits       13917   13916      -1     
- Misses      7701    7702      +1     
Impacted Files Coverage Δ
light-client/src/lib.rs 100.0% <ø> (ø)
proto/src/lib.rs 100.0% <ø> (ø)
tendermint/src/lib.rs 100.0% <ø> (ø)
testgen/src/vote.rs 84.1% <0.0%> (-0.9%) ⬇️
testgen/src/header.rs 83.5% <0.0%> (-0.6%) ⬇️
abci/src/server.rs 9.4% <0.0%> (+0.1%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@thanethomson
Copy link
Contributor

re: release.sh do I have the appropriate permissions to push to crates.io once this PR gets approved?

No, I've just pushed #1268 to attempt to help with this. We've configured our @bftbot to be able to push crates on our behalf when you push tags to the repo. Please double-check my approach in #1268 to see if I've missed anything important?

If we can merge #1268 first, then this PR, then if you push the v0.29.0 tag to the repo it should trigger the release. Please also sign and annotate the tag before pushing it:

git tag -s -a v0.29.0 -m 'Release v0.29.0'

I usually manually create a release from a new tag. Eventually we'll automate that too.

Note: if you’re using docs.rs, setting html_root_url manually is obsolete: rust-lang/api-guidelines#229

Thanks for the heads-up @tarcieri!

@@ -183,8 +183,6 @@ Our release process is as follows:
changes to the release PR. This includes:
1. All `Cargo.toml` files (making sure dependencies' versions are updated
too).
2. All crates' `lib.rs` files documentation references' `html_root_url`
Copy link
Contributor

Choose a reason for hiding this comment

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

Please edit point (3) above to make it a single sentence.

@@ -0,0 +1,3 @@
*Feb 17, 2023*

This release features modularity improvements for the cryptographic routines, as well as fixes related to block verification and the use of consensus-friendly ed25519 crate.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This release features modularity improvements for the cryptographic routines, as well as fixes related to block verification and the use of consensus-friendly ed25519 crate.
This release features modularity improvements for the cryptographic routines, as well as fixes related to block verification and the use of a consensus-friendly ed25519 crate.

Copy link
Contributor

Choose a reason for hiding this comment

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

To match the generated changelog.

Copy link
Contributor

@thanethomson thanethomson left a comment

Choose a reason for hiding this comment

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

🎉

@erwanor erwanor merged commit 390cc90 into main Feb 17, 2023
@erwanor erwanor deleted the release/v0.29.0 branch February 17, 2023 22:28
@erwanor erwanor mentioned this pull request Feb 21, 2023
7 tasks
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.

Release an 0.29 with #1245 included
4 participants