-
-
Notifications
You must be signed in to change notification settings - Fork 943
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
refactor!: cleanup deprecations #1440
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## next #1440 +/- ##
==========================================
- Coverage 99.62% 99.61% -0.02%
==========================================
Files 2166 2164 -2
Lines 237450 236816 -634
Branches 1044 1001 -43
==========================================
- Hits 236567 235898 -669
- Misses 862 896 +34
- Partials 21 22 +1
|
Co-authored-by: ST-DDT <ST-DDT@gmx.de>
Co-authored-by: ST-DDT <ST-DDT@gmx.de>
How do we handle this as it is marked for removal in v8.0? faker/src/modules/address/index.ts Lines 137 to 152 in 2914a69
|
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.
You missed this on:
faker/src/modules/datatype/index.ts
Lines 220 to 235 in 2914a69
hexadecimal( | |
options: | |
| { length?: number; prefix?: string; case?: 'lower' | 'upper' | 'mixed' } | |
| number = {} | |
): string { | |
if (typeof options === 'number') { | |
deprecated({ | |
deprecated: 'faker.datatype.hexadecimal(length)', | |
proposed: 'faker.datatype.hexadecimal({ length })', | |
since: '7.5', | |
until: '8.0', | |
}); | |
options = { | |
length: options, | |
}; | |
} |
Good catch. |
@xDivisionByZerox nice catch! |
closes #1439