Skip to content
This repository has been archived by the owner on Dec 18, 2024. It is now read-only.

Update tonic #579

Closed
wants to merge 1 commit into from
Closed

Conversation

argerus
Copy link
Contributor

@argerus argerus commented Jun 16, 2023

No description provided.

@argerus argerus force-pushed the topic/update_tonic branch from c3d48c2 to cd6f155 Compare June 16, 2023 13:33
@erikbosch
Copy link
Contributor

The failure on test is interesting. I have seen it before, now and then. I do not really know how we specify cargo/protoc-version to use. Is it possibly so that we use "latest and greatest" which often but not always is a nightly build. I retriggered the build.

error: the option Z is only accepted on the nightly compiler

@erikbosch
Copy link
Contributor

Works well locally for me, but I would prefer if we could solve the build regression on master so that build can pass here before merging

@argerus
Copy link
Contributor Author

argerus commented Jun 20, 2023

Will be included in #580 instead

@argerus argerus closed this Jun 20, 2023
@erikbosch erikbosch deleted the topic/update_tonic branch October 31, 2024 12:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants