Skip to content

feat(locale): add Welsh (cy) city name definitions#3884

Open
IestynGage wants to merge 1 commit into
faker-js:nextfrom
IestynGage:next
Open

feat(locale): add Welsh (cy) city name definitions#3884
IestynGage wants to merge 1 commit into
faker-js:nextfrom
IestynGage:next

Conversation

@IestynGage
Copy link
Copy Markdown
Contributor

I contemplated using real welsh city names, however they are only 7 and they overlap names with the counties.
i.e. Abertawe, Caerdydd, Casnewydd and Wrecsam, so for the address method it could return the same city + county.

So as a alternative, I used city_prefix and city_suffix to make fake welsh sounding cities. Similiar to en_GB.

I think a welsh speak would know they are fake, but they look like welsh cities names.

I got the prefix & suffix by looking at welsh cities/towns/villages and breaking them up. Like how you can see a lot of welsh towns named Aber-something. (Aber meaning 'river mouth' or where the river meets the sea, so Aberystwyth = where the river ystwyth meets the sea).

I contemplated using the welsh city names,
however they are only 7 and they overlap names with the counties.

i.e. Abertawe, Caerdydd, Casnewydd and Wrecsam

So as a alternative,
I used `city_prefix` and `city_suffix` to make fake welsh sounding cities.
Similiar to `en_GB`.

I think a welsh speak would know they are fake,
but they look like welsh cities names.

I got the prefix & suffix by looking at welsh cities/towns/villages and breaking them up.
Like how you can see a lot of welsh towns named `Aber`-something.
@IestynGage IestynGage requested a review from a team as a code owner June 5, 2026 22:12
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 5, 2026

Deploy Preview for fakerjs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 07bc3d7
🔍 Latest deploy log https://app.netlify.com/projects/fakerjs/deploys/6a2349b6e7899d000970a9cd
😎 Deploy Preview https://deploy-preview-3884.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.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.92%. Comparing base (03a1682) to head (07bc3d7).

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #3884   +/-   ##
=======================================
  Coverage   98.92%   98.92%           
=======================================
  Files         910      910           
  Lines        3161     3161           
  Branches      581      581           
=======================================
  Hits         3127     3127           
  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.

@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 Jun 6, 2026
@ST-DDT ST-DDT added this to the v10.x milestone Jun 6, 2026
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.

3 participants