feat(locales): add sl_SI locale, person module

[ZigaVukcevicDev]

No description provided.

[netlify]

✅ Deploy Preview for fakerjs ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	bedddc5
🔍 Latest deploy log	https://app.netlify.com/projects/fakerjs/deploys/686d5dbee20f570008e0ef16
😎 Deploy Preview	https://deploy-preview-3564.fakerjs.dev
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[matthewmayer]

Thanks!

Note for the initial PR for adding a language we recommend adding one module only (e.g. person as you have done here). Please don't add additional modules to this PR until it has been reviewed and approved. Once it's approved you can follow up with additional PRs to add further modules.

Note also we are soon releasing v10, a major release, so we likely won't merge this imminently. It is more likely to make it into a v10.1 release in a few weeks.

[matthewmayer]

You are currently missing the first_name and last_name files.

[ZigaVukcevicDev]

@matthewmayer I know they were missing, this is why PR was set to draft. Thank you for the notice though.

I have added them now.

[ZigaVukcevicDev]

I tried to run pnpm run preflight but ran into errors

I am asking for some assistance on how to approach this issue, thank you.

[Shinigami92]

I tried to run pnpm run preflight but ran into errors

I am asking for some assistance on how to approach this issue, thank you.

I'll run the CI and so we can identify if it's a real error or just a local one to your environment

[Shinigami92]

I assume it has todo something with that suffix is null in this PR
Can't check right now immediately cause I'm on mobile

[ZigaVukcevicDev]

@Shinigami92 I have updated file src/locales/sl_SI/person/suffix.ts with array of values instead of null and problem is solved.

[ZigaVukcevicDev]

Ready for review. Thank you.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.96%. Comparing base (93416f7) to head (bedddc5).

Additional details and impacted files

@@           Coverage Diff            @@
##             next    #3564    +/-   ##
========================================
  Coverage   99.96%   99.96%            
========================================
  Files        2888     2908    +20     
  Lines      221958   222794   +836     
  Branches      930      928     -2     
========================================
+ Hits       221889   222725   +836     
  Misses         69       69            

Files with missing lines	Coverage Δ
src/locale/index.ts	100.00% <100.00%> (ø)
src/locale/sl_SI.ts	100.00% <100.00%> (ø)
src/locales/index.ts	100.00% <100.00%> (ø)
src/locales/sl_SI/index.ts	100.00% <100.00%> (ø)
src/locales/sl_SI/metadata.ts	100.00% <100.00%> (ø)
src/locales/sl_SI/person/bio_part.ts	100.00% <100.00%> (ø)
src/locales/sl_SI/person/bio_pattern.ts	100.00% <100.00%> (ø)
src/locales/sl_SI/person/bio_supporter.ts	100.00% <100.00%> (ø)
src/locales/sl_SI/person/first_name.ts	100.00% <100.00%> (ø)
src/locales/sl_SI/person/index.ts	100.00% <100.00%> (ø)
... and 12 more

🚀 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]

For future reference, If you don't want to provide suffixes for a locale, thats fine (not all languages have them) but then you need to add the locale to line 61 of test/all-functional.spec.ts to allow the test to pass