-
-
Notifications
You must be signed in to change notification settings - Fork 934
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
fix(locale): improve Swedish phone numbers format #2520
Conversation
https://en.wikipedia.org/wiki/Telephone_numbers_in_Sweden Numbers starting with 070, 072, 073, 076 or 079 are reserved for mobile numbers in Sweden.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## next #2520 +/- ##
=======================================
Coverage 99.57% 99.58%
=======================================
Files 2779 2779
Lines 249273 249303 +30
Branches 1080 1083 +3
=======================================
+ Hits 248217 248265 +48
+ Misses 1028 1010 -18
Partials 28 28
|
Note that faker.phone.number() does not specifically promise to return mobile numbers. Can also be landlines. If you have a use-case for mobile numbers only it might be good to get your input on #1542 |
That's true, I could add formats for landline numbers as well. But the current formats does not look like any Swedish numbers. So I thought that this is at least make the SV faker version return Swedish phone number (but not all kinds). Will write in the other issue as well, but in the meantime, is this change something you would be interested in or how can I change it to follow the guidelines? Add these formats to the cell formats instead? |
If it's an improvement from the current data we should accept this PR anyway. At the moment the cell_phone definitions are not actively used by Faker. You can have a read through the discussion on the other issue and add any comments on how you would typically use this data to help guide us on what would be most useful in future. |
I think it would be best to include them for now so that we have them already when we split/improve the method |
And please also include ones with the international prefix for variation. |
https://en.wikipedia.org/wiki/Telephone_numbers_in_Sweden
Numbers starting with 070, 072, 073, 076 or 079 are reserved for mobile numbers in Sweden.