-
Notifications
You must be signed in to change notification settings - Fork 4
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
Fixes & updates #2
Conversation
LGTM /cc @louy. Would you be interested in helping maintain the project? I can add you as a collaborator if you are 😄 |
@blakeembrey, I'd be glad to 😄 |
@IOAyman Great! I've added you, just try to stick to PRs still so anyone else can jump in 😄 Edit: Also, use squashing 😄 |
@IOAyman There's also the version in https://github.com/types/npm-validator/blob/master/typings.json which will usually need updating. I created some basic guidelines in https://github.com/types/guidelines, not sure how useful that is for you. |
@blakeembrey, Thnaks! I'll update the remaining file, and squash things 😉 |
isAfter
andisBefore
: The 2nd arg should be astring
isMobilePhone
typings
toolisDataURI
,isMD5
validators andunescape
sanitizertoString