feat(locale): support street and street address for ja

[KeisukeYamashita]

Fixed (added) Japanese address output.

Previously, English addresses were generated incorrectly, but the format has been changed to the correct address format.
Street address example:

import { fakerJA as faker } from "@faker-js/faker";

// Before
faker.location.streetAddress() // 5668 杏Square. Incorrect
faker.location.streetAddress(true) // 5668 杏Square. Incorrect

// After
faker.location.streetAddress() // 8丁目7番4号. Correct
faker.location.streetAddress(true) // 9丁目2番3号 木村768号室. Correct

Symbols can now be used in street addresses because street names in Japan sometimes contain numbers, but this has no other destructive effect. Of course, not all country road formats include numbers, but I believe they are used in many countries. I supported it because the use of symbols is completely restricted now.

Thank you in advance.

Disclaimer

I am not a government official or lawyer in charge of Japanese geography. However, in its current state, it is difficult to use it for address dummy data, so a PR was created.

[KeisukeYamashita]

Sorry, the CI failed.
Let me check 🙏

[ST-DDT]

Looks like you forgot to run pnpm run preflight 😉 .

[matthewmayer]

https://en.wikipedia.org/wiki/Japanese_addressing_system

[KeisukeYamashita]

Looks like you forgot to run pnpm run preflight 😉 .

Thank you very much for your comment, I have ran and push this commit 👉 a83f75a

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.93%. Comparing base (f47dfe5) to head (76a8343).

Additional details and impacted files

@@           Coverage Diff           @@
##             next    #2193   +/-   ##
=======================================
  Coverage   99.93%   99.93%           
=======================================
  Files        2973     2975    +2     
  Lines      212592   212600    +8     
  Branches      949      951    +2     
=======================================
+ Hits       212452   212460    +8     
- Misses        136      140    +4     
+ Partials        4        0    -4     

Files	Coverage Δ
src/locales/ja/location/index.ts	100.00% <100.00%> (ø)
src/locales/ja/location/secondary_address.ts	100.00% <100.00%> (ø)
src/locales/ja/location/street_address.ts	100.00% <100.00%> (ø)
src/locales/ja/location/street_pattern.ts	100.00% <100.00%> (ø)
src/modules/location/index.ts	99.67% <100.00%> (ø)

... and 5 files with indirect coverage changes

Wrong review.

[ST-DDT]

Sorry for the long wait. This somehow got lost in the PR backlog.
I think we have to discuss internally if the fake pattern usage here is what we want.

[ST-DDT]

Team Decision

• We accept this change. Please rebase so that we can merge this.

[netlify]

✅ Deploy Preview for fakerjs ready!

Name	Link
🔨 Latest commit	76a8343
🔍 Latest deploy log	https://app.netlify.com/sites/fakerjs/deploys/66118bc837a3a000085be2ae
😎 Deploy Preview	https://deploy-preview-2193.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 site configuration.

[KeisukeYamashita]

Thank you for your consideration!
I've fixed the conflict and updated the main branch. Thanks in advance.