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

Fix generated validation code for gRPC #3409

Merged
merged 2 commits into from
Nov 6, 2023

Conversation

tchssk
Copy link
Member

@tchssk tchssk commented Nov 5, 2023

The protobuf types do not consider Meta, so the validation code should not too.

@tchssk tchssk changed the title Add test cases of struct:field:name for gRPC Fix generated validation code for gRPC Nov 5, 2023
@tchssk tchssk marked this pull request as ready for review November 5, 2023 12:01
@raphael
Copy link
Member

raphael commented Nov 6, 2023

This is really great, thank you for the nice fix and tests!

@raphael raphael merged commit 42a9ddd into goadesign:v3 Nov 6, 2023
7 checks passed
@tchssk tchssk deleted the grpc-codegen-validations-remove-meta branch November 6, 2023 05:47
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