Skip to content
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

If fixed_line/mobile validators same, is either. #85

Merged
merged 1 commit into from
Dec 2, 2017

Conversation

dracos
Copy link
Contributor

@dracos dracos commented Oct 5, 2017

This matches libphonenumber behaviour of using FIXED_LINE_OR_MOBILE in
such a circumstance, and matches the existing behaviour with US numbers.

Fixes #84.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.967% when pulling 1ba02fb on dracos:84-fixed-mobile-regex-same into 850ad31 on DrHyde:master.

This matches libphonenumber behaviour of using FIXED_LINE_OR_MOBILE in
such a circumstance, and matches the existing behaviour with US numbers.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.967% when pulling 02f12d4 on dracos:84-fixed-mobile-regex-same into 850ad31 on DrHyde:master.

@DrHyde DrHyde merged commit e8b6d1a into DrHyde:master Dec 2, 2017
@DrHyde
Copy link
Owner

DrHyde commented Dec 2, 2017

Thanks, merged

@dracos dracos deleted the 84-fixed-mobile-regex-same branch January 25, 2018 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants