-
Notifications
You must be signed in to change notification settings - Fork 408
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
✨ add min/max properties validation #488
✨ add min/max properties validation #488
Conversation
Welcome @moolen! |
4781e50
to
c23e2dc
Compare
/assign @vincepri |
Can we add a test? |
c23e2dc
to
b99995b
Compare
Hey @vincepri sure, that totally makes sense! I missed that on the first glance (i was looking for |
/approve |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: DirectXMan12, moolen, vincepri The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This PR adds
minProperties
/maxProperties
validation markers.fixes #487