-
-
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
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