-
Notifications
You must be signed in to change notification settings - Fork 3
Support for Indian phone numbers #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
Conversation
Looks good to me, @johnotander thoughts? Seems like i can ping only you of all the commiters (sindre and arthur) ;d don't ask why. @jonschlinkert does the regex feels good, you-guru-man? 🍰 😆 |
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.
LGTM!
@johnotander just a reminder :) |
Lgtm. Won't be at my computer today to publish, perhaps @charlike can bump
a minor and publish?
…On September 17, 2017 at 1:24:46 AM, Livin ***@***.***) wrote:
@johnotander <https://github.com/johnotander> just a reminder :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABW8vS85MMPw6pHqo7c3bIrCTsdNXRnSks5sjMk-gaJpZM4PPE33>
.
|
I can merge and tag it, but can't publish since i don't have npm rights for it :) |
Added you as an owner 💖 |
Great. I'll release minor bump. |
Done! :) |
Added support for indian phone numbers