-
Notifications
You must be signed in to change notification settings - Fork 26
Migrated to nnbd #24
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
base: master
Are you sure you want to change the base?
Migrated to nnbd #24
Conversation
I've published validators2 to pub.dev as a nnbd implementation until the authors have time to publish one of the two outstanding PRs for nnbd. |
@bsutton It seems like after your recent commits this PR couldn't be merged into this repo anymore (as now it has conflicts, and also changes the library name to `validators2). I'd suggest creating a new PR from your commit 091130e. And thanks for your job, I really need a null safe version of this library. |
resolved the merge conflicts and fixed the package name. |
@bsutton A new version got published several days ago, that supports null safety. |
There are a number of small breaking changes, the main theme is changing optional args to named args with a default value.
…ml from published files.
The vin library has released an nnbd version so I've migrated validators to nndb (given that 2.12 is now released).
Can you please publish as a preview.
Thanks.