Skip to content

chore(locale)!: remove legacy cell_phone definitions#3925

Draft
matthewmayer wants to merge 5 commits into
faker-js:nextfrom
matthewmayer:remove-cellphone
Draft

chore(locale)!: remove legacy cell_phone definitions#3925
matthewmayer wants to merge 5 commits into
faker-js:nextfrom
matthewmayer:remove-cellphone

Conversation

@matthewmayer

Copy link
Copy Markdown
Contributor

Recommend to use faker.phone.number({style:"mobile"}) instead.

@netlify

netlify Bot commented Jun 29, 2026

Copy link
Copy Markdown

Deploy Preview for fakerjs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 3714904
🔍 Latest deploy log https://app.netlify.com/projects/fakerjs/deploys/6a435f3973cf2f0008b9ad07
😎 Deploy Preview https://deploy-preview-3925.fakerjs.dev
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@matthewmayer matthewmayer added the breaking change Cannot be merged when next version is not a major release label Jun 29, 2026
@matthewmayer matthewmayer added this to the v11.0 milestone Jun 29, 2026
ST-DDT
ST-DDT previously approved these changes Jun 29, 2026

@ST-DDT ST-DDT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can merge this cleanup in v11.0.
Though we might want to also remove the other legacy data (in a separate PR). As nobody seems to be requesting them as a feature.

@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.77%. Comparing base (acd5fda) to head (3714904).

Additional details and impacted files
@@            Coverage Diff             @@
##             next    #3925      +/-   ##
==========================================
- Coverage   98.84%   98.77%   -0.08%     
==========================================
  Files         923      890      -33     
  Lines        3216     3183      -33     
  Branches      583      583              
==========================================
- Hits         3179     3144      -35     
- Misses         33       35       +2     
  Partials        4        4              
Files with missing lines Coverage Δ
src/locales/af_ZA/index.ts 100.00% <ø> (ø)
src/locales/ar/index.ts 100.00% <ø> (ø)
src/locales/de/index.ts 100.00% <ø> (ø)
src/locales/de_AT/index.ts 100.00% <ø> (ø)
src/locales/dv/index.ts 100.00% <ø> (ø)
src/locales/el/index.ts 100.00% <ø> (ø)
src/locales/en/index.ts 100.00% <ø> (ø)
src/locales/en_GB/index.ts 100.00% <ø> (ø)
src/locales/en_IE/index.ts 100.00% <ø> (ø)
src/locales/en_ZA/index.ts 100.00% <ø> (ø)
... and 23 more

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ST-DDT ST-DDT added p: 1-normal Nothing urgent c: locale Permutes locale definitions labels Jun 29, 2026
Comment thread docs/guide/upgrading_v11/3925.md
Comment thread docs/guide/upgrading_v11/3925.md Outdated
matthewmayer and others added 2 commits June 30, 2026 13:14
Co-authored-by: ST-DDT <ST-DDT@gmx.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Cannot be merged when next version is not a major release c: locale Permutes locale definitions p: 1-normal Nothing urgent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants