Skip to content

feat(locale): add Welsh (cy) postal address definitions#3850

Merged
ST-DDT merged 1 commit into
faker-js:nextfrom
IestynGage:postal_address
May 19, 2026
Merged

feat(locale): add Welsh (cy) postal address definitions#3850
ST-DDT merged 1 commit into
faker-js:nextfrom
IestynGage:postal_address

Conversation

@IestynGage
Copy link
Copy Markdown
Contributor

Follows #3849.

Postal address definitions as ask by @matthewmayer in #3849 (comment). Uses the new county, based on the EN_GB address.

@IestynGage IestynGage requested a review from a team as a code owner May 15, 2026 19:39
@netlify
Copy link
Copy Markdown

netlify Bot commented May 15, 2026

Deploy Preview for fakerjs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 9c5d5d8
🔍 Latest deploy log https://app.netlify.com/projects/fakerjs/deploys/6a0b722ae7ad51000893640f
😎 Deploy Preview https://deploy-preview-3850.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.

@IestynGage IestynGage changed the title Add Welsh (cy) postal address definitions add Welsh (cy) postal address definitions May 15, 2026
@IestynGage IestynGage changed the title add Welsh (cy) postal address definitions feat(locale): add Welsh (cy) postal address definitions May 15, 2026
IestynGage added a commit to IestynGage/faker that referenced this pull request May 15, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.91%. Comparing base (f51d28a) to head (9c5d5d8).

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #3850   +/-   ##
=======================================
  Coverage   98.91%   98.91%           
=======================================
  Files         905      905           
  Lines        3146     3146           
  Branches      564      581   +17     
=======================================
  Hits         3112     3112           
  Misses         30       30           
  Partials        4        4           
Files with missing lines Coverage Δ
src/locales/cy/location/index.ts 100.00% <ø> (ø)
🚀 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.

Shinigami92
Shinigami92 previously approved these changes May 18, 2026
@ST-DDT ST-DDT added p: 1-normal Nothing urgent c: locale Permutes locale definitions m: location Something is referring to the location module labels May 18, 2026
@ST-DDT ST-DDT added this to the v10.x milestone May 18, 2026
@ST-DDT
Copy link
Copy Markdown
Member

ST-DDT commented May 18, 2026

There seems to be merge conflicts.

@ST-DDT ST-DDT requested review from a team May 18, 2026 22:14
@matthewmayer
Copy link
Copy Markdown
Contributor

LGTM

Current output is something like:

69201 Wellington Street
North Jaydonbury
Sir Benfro
33132-3663

Note US style ZIP code. To make this more realistic, in another PR you could copy in the en_GB postcode implementation.

https://github.com/faker-js/faker/blob/next/src/locales/en_GB/location/postcode.ts

Or even better, include the relevant Welsh postcode areas in the patterns

CF
LD
LL
NP
SA
SY?
CH?

@ST-DDT ST-DDT added this pull request to the merge queue May 19, 2026
Merged via the queue into faker-js:next with commit e72fd7c May 19, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: locale Permutes locale definitions m: location Something is referring to the location module p: 1-normal Nothing urgent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants