-
Notifications
You must be signed in to change notification settings - Fork 341
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
Validate attributes are missing on PositiveInt #2426
Labels
Comments
We probably should, but it would not fix #2424, because it will not change the behaviour of the constructor, only of the validator! |
I removed this from the sprint, because it does not solve the issue #2424, and it's located in a remote area of the SDK, which is hardly used. There are more important issues to pick up. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Do we not need validate attributes on
PositiveInt
? This would solve #2424.The text was updated successfully, but these errors were encountered: