-
Notifications
You must be signed in to change notification settings - Fork 508
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
Bug on codegen in ::prost::Enumeration #927
Comments
Bump on this @LucioFranco or another maintainer. Would be glad to pitch in here, it's preventing a major version update for anyone using enums. |
May I kindly inquire about the latest developments regarding this issue and if there are any temporary solutions available at the moment? |
Using the previous minor version for now. |
@LucioFranco or other maintainer, any desire for a fix here? |
Hi @0xAlcibiades sorry about the delay. I don't have time right now to come up with a fix but I can review a PR if someone is interested in picking up that work. |
@LucioFranco Yeah, I think it'll involve reverting the changes, or fixing the changes from #853 |
@LucioFranco @megsdevs put in PR #955 to resolve this issue. |
Using tonic 0.10.1 and prost-types/prost 0.12.1, on building codegen for proto with an enumeration defined:
I see this bug, which prevents build from succeeding. This appears to be a regression in 0.9.2->0.10.1 tonic and 0.11.9 prost.
This was also noted in #812 in a recent comment.
The text was updated successfully, but these errors were encountered: