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
Remove unused Validations code #2103
Conversation
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.
This LGTM, thanks :)
It seems Git cannot figure out the merge since the commits from the branch you targeted were squashed into a single commit, sorry about that. You should be able to cherry pick the commit here onto master.
@tanner0101 I meant to use tn-core-service as a base. Now it should be ok 🙂 |
- remove ValidatorType - avoid force unwrap - avoid illegal count and range operators by not exposing .range() and .count()
7f6def8
to
c95dd0e
Compare
@tanner0101 I misunderstood, you had already merged tn-core-service into master. I was able to cherry pick my changes and force push so now it should be ok 🙂 |
These changes are now available in 4.0.0-rc.2 |
- remove ValidatorType - avoid force unwrap - avoid illegal count and range operators by not exposing .range() and .count()
This is technically a breaking change since some public API is being removed, but it is not being used anywhere.
ValidatorType