-
Notifications
You must be signed in to change notification settings - Fork 10
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.
Yoo thanks, this looks really nice. Would you mind adding this new func to docs on README after that this will get merged and published on npm
Task finished . |
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.
Thanks
CI failed https://github.com/veritem/valdie/actions/runs/3886614305/jobs/6632439445 would you mind looking into why that test failed? |
fixed all problems . |
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.
Thanks
This PR mplements the following feature (isDistinct)
Closing issues
Fixes #101