Skip to content

Validator generator [WiP]#10200

Draft
stephanos wants to merge 1 commit into
mainfrom
stephanos/protobuf-validate-gen
Draft

Validator generator [WiP]#10200
stephanos wants to merge 1 commit into
mainfrom
stephanos/protobuf-validate-gen

Conversation

@stephanos
Copy link
Copy Markdown
Contributor

What changed?

Describe what has changed in this PR.

Why?

Tell your future self why have you made these changes.

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)

Potential risks

Any change is risky. Identify all risks you are aware of. If none, remove this section.

@stephanos stephanos force-pushed the stephanos/protobuf-validate-gen branch 6 times, most recently from 0fc7732 to e10b967 Compare May 8, 2026 02:43
Comment thread tests/validators_test.go
"go.uber.org/fx"
)

type ValidatorsSuite struct {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test detects if a validator has a missing field validator.

@stephanos stephanos force-pushed the stephanos/protobuf-validate-gen branch 4 times, most recently from 6e2ce05 to 637a3c3 Compare May 8, 2026 02:48
@stephanos stephanos force-pushed the stephanos/protobuf-validate-gen branch from 637a3c3 to c42f6af Compare May 8, 2026 02:49
@long-nt-tran long-nt-tran self-requested a review May 8, 2026 18:46
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.

1 participant