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

prepare v0.19.0, bump tower dep to v0.5 #59

Merged
merged 3 commits into from
Feb 4, 2025

Conversation

SuperFluffy
Copy link
Contributor

Bumps tower to v0.5 to match the rest of the ecosystem (tonic, axum).

@@ -17,10 +17,10 @@ documentation = "https://docs.rs/tower-abci"
tendermint-proto = "0.40"
tendermint = "0.40"
bytes = "1"
tokio = { version = "1", features = ["full"]}
Copy link
Member

Choose a reason for hiding this comment

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

Paged out the details of why, but iirc this is needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I can move this change to a separate PR if you want - but the features are not fully needed (for both tokio and tower). More of the features are needed for the examples to work, but not the full set.

Reason to not have this is to not pull in the entire kitchen sink when importing tower-abci.

Copy link
Member

Choose a reason for hiding this comment

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

No need, sounds good. Thx

@erwanor
Copy link
Member

erwanor commented Feb 4, 2025

ACK, will merge on green CI, do you need a new release as well?

@SuperFluffy
Copy link
Contributor Author

@erwanor CI should pass now. The change in features is technically not best practice in PRs, but I figured it's minimal and not observable outside of tower-abci.

Let me know if you want me to move this to a separate PR.

And yes, a release would be nice.

@SuperFluffy SuperFluffy requested a review from erwanor February 4, 2025 18:59
@erwanor
Copy link
Member

erwanor commented Feb 4, 2025

Can you bump the crate version in the PR pls, i will squash and release from the UI. afk

@SuperFluffy SuperFluffy changed the title bump [email protected] prepare v0.19.0, bump tower dep to v0.5 Feb 4, 2025
@SuperFluffy
Copy link
Contributor Author

Can you bump the crate version in the PR pls, i will squash and release from the UI. afk

Updated the version and the PR title

@erwanor erwanor merged commit 0e57406 into penumbra-zone:main Feb 4, 2025
4 checks passed
erwanor pushed a commit that referenced this pull request Feb 5, 2025
Bumps tokio-util to 0.7. Similar to PR #59, this sets the version to
0.19.1.

I went through all imports of tokio_utils and it looks like none of it
is part of the crate's public API, making this a minor release. The only
`pub` marked items are the `v{034,037,038}::codex::{Decode}` types,
which `impl tokio_util::codec::Decoder` and `impl
tokio_util::codec::Decoder`. But the crate does not actually export any
of these items (and the `*::codex` modules stay private).
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.

2 participants