-
-
Notifications
You must be signed in to change notification settings - Fork 953
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
feat(phone)!: add new style parameter #2578
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## next #2578 +/- ##
==========================================
- Coverage 99.57% 99.57% -0.01%
==========================================
Files 2859 3034 +175
Lines 248602 250972 +2370
Branches 985 994 +9
==========================================
+ Hits 247541 249900 +2359
- Misses 1061 1072 +11
|
We'll need rebase after #2712 lands |
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.
I forgot this had a pending comment...
Co-authored-by: ST-DDT <ST-DDT@gmx.de>
Co-authored-by: Eric Cheng <ericcheng9316@gmail.com>
The description states it is still WIP. Is that outdated or what is left to do? |
Nope sorry not WIP any more. Ready for final review. |
I am sorry but I am lost, using parameters with faker.phone.number() with parameters is deprecated. So why this PR seems to add parameters ? Is the deprecation canceled or something ? |
The new parameters are entirely different and incompatible with the previous ones. |
In which version will it be available ? |
v9.0.0 (+preview releases) We plan to release an alpha soon. Maybe next week? |
fix #1542
Script for doing the grunt work of renaming the formats.ts and generating the initial raw and national files