-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: Add protoc plugin to generate tests #122
Conversation
decd773
to
eaa0918
Compare
eaa0918
to
1dac0c5
Compare
1dac0c5
to
01c7451
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks very reasonable to me!
@blakeli0 you might be interested in taking a look, as this is a mini-generator that's using protoc directly. There's also use of Velocity templates to generate tests.
# --experimental_allow_proto3_optional \ | ||
# $proto_src | ||
echo "# - validation tests" | ||
$PROTOC_PATH \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you considered using the Xolstice protobuf-maven-plugin?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, we are trying to minimize any additional dependencies and rely on tools that have long term maintenance plans.
No description provided.