Skip to content

feat(locale): add sl_SI locale, location module#3652

Open
ZigaVukcevicDev wants to merge 33 commits intofaker-js:nextfrom
ZigaVukcevicDev:feat/locale-sl-SI-location
Open

feat(locale): add sl_SI locale, location module#3652
ZigaVukcevicDev wants to merge 33 commits intofaker-js:nextfrom
ZigaVukcevicDev:feat/locale-sl-SI-location

Conversation

@ZigaVukcevicDev
Copy link
Copy Markdown
Contributor

No description provided.

@ZigaVukcevicDev ZigaVukcevicDev requested a review from a team as a code owner November 6, 2025 15:33
@netlify
Copy link
Copy Markdown

netlify bot commented Nov 6, 2025

Deploy Preview for fakerjs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit a65b03d
🔍 Latest deploy log https://app.netlify.com/projects/fakerjs/deploys/69d27afc41f78e0008f41c08
😎 Deploy Preview https://deploy-preview-3652.fakerjs.dev
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.88%. Comparing base (5e1ce98) to head (a65b03d).

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #3652   +/-   ##
=======================================
  Coverage   98.88%   98.88%           
=======================================
  Files         886      887    +1     
  Lines        3061     3062    +1     
  Branches      540      556   +16     
=======================================
+ Hits         3027     3028    +1     
  Misses         30       30           
  Partials        4        4           
Files with missing lines Coverage Δ
src/locales/sl_SI/index.ts 100.00% <ø> (ø)
src/locales/sl_SI/location/index.ts 100.00% <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.

matthewmayer
matthewmayer previously approved these changes Nov 13, 2025
@xDivisionByZerox xDivisionByZerox added c: feature Request for new feature c: locale Permutes locale definitions m: location Something is referring to the location module labels Dec 14, 2025
@xDivisionByZerox xDivisionByZerox added this to the v10.x milestone Dec 14, 2025
Copy link
Copy Markdown
Member

@ST-DDT ST-DDT left a comment

Choose a reason for hiding this comment

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

Sorry for the slow review.

I don't speak Slovenian, so my suggestions are just what I found during the review.
Could you please check whether your version/the suggestion is correct?

@ST-DDT
Copy link
Copy Markdown
Member

ST-DDT commented Mar 31, 2026

@ZigaVukcevicDev Can you please check my comments?

@ZigaVukcevicDev
Copy link
Copy Markdown
Contributor Author

@ST-DDT I will check it in couple of days. Thank you.

@ZigaVukcevicDev
Copy link
Copy Markdown
Contributor Author

@ST-DDT comments resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: feature Request for new feature c: locale Permutes locale definitions m: location Something is referring to the location module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants