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

Bump prost to 0.11.0 and tonic to 0.8 #73

Merged
merged 2 commits into from
Sep 12, 2022
Merged

Conversation

sd2k
Copy link
Collaborator

@sd2k sd2k commented Aug 2, 2022

This also removes the build dependencies on prost-build and tonic-build
because of the new requirement for protoc to be installed. Instead,
the generated code is committed as part of this PR, with instructions on
how to regenerate it in Cargo.toml. The build script is still present
but disabled using build = false in Cargo.toml.

This should speed up build times and reduces our dependencies, which is
great!

@sd2k
Copy link
Collaborator Author

sd2k commented Aug 21, 2022

Marking as draft until hyperium/tonic#1065 is merged and released, and I can remove the patches.

@sd2k sd2k marked this pull request as draft August 21, 2022 13:11
@sd2k sd2k force-pushed the bump-prost-0.11-tonic-0.8 branch from e397002 to bd70dc5 Compare August 22, 2022 13:58
This also removes the build dependencies on prost-build and tonic-build
because of the new requirement for `protoc` to be installed. Instead,
the generated code is committed as part of this PR, with instructions on
how to regenerate it in Cargo.toml. The build script is still present
but disabled using `build = false` in Cargo.toml.

This should speed up build times and reduces our dependencies, which is
great!
@sd2k sd2k marked this pull request as ready for review September 12, 2022 09:14
@sd2k sd2k merged commit 2f03c98 into main Sep 12, 2022
@sd2k sd2k deleted the bump-prost-0.11-tonic-0.8 branch September 12, 2022 09:15
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

1 participant