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

chore(deps): bump tonic to 0.11 and raise MSRV to 1.70 #547

Merged
merged 2 commits into from
May 16, 2024

Conversation

matze
Copy link
Contributor

@matze matze commented Apr 19, 2024

No description provided.

@matze matze requested a review from a team as a code owner April 19, 2024 15:39
Copy link
Collaborator

@hds hds left a comment

Choose a reason for hiding this comment

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

By bumping to a semver incompatible version of tonic, this change needs to be marked as incompatible in the commit message (https://www.conventionalcommits.org/en/v1.0.0/).

Also, the console-api protobuf files need to be updated, as do the ones console-subscriber grpc web example.

For console-api the following command needs to be run:

cargo run -p xtask -- gen-proto

For the example, the instructions are here:
https://github.com/tokio-rs/console/tree/main/console-subscriber/examples/grpc_web#understanding-the-code

console-api/Cargo.toml Show resolved Hide resolved
BREAKING CHANGE:
This is a breaking change for users of `console-api` and
`console-subscriber`, as it changes the public `tonic` dependency to a
semver-incompatible version. This breaks compatibility with `tonic`
0.10.x.
@Xuanwo
Copy link

Xuanwo commented May 11, 2024

Hello, I came across this PR because console-api and console-subscriber are blocking our project's upgrade to tonic 0.11. Is this PR ready for merge?

Copy link
Collaborator

@hi-rustin hi-rustin left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks! 👍

Blocked by #515

@hi-rustin
Copy link
Collaborator

@hds Could you please help approve this PR? Then we can merge it. Thanks!

@hi-rustin hi-rustin requested a review from hds May 16, 2024 13:08
Copy link
Collaborator

@hds hds left a comment

Choose a reason for hiding this comment

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

Thanks for adding this one!

@hds hds merged commit ef6816c into tokio-rs:main May 16, 2024
17 checks passed
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

4 participants