-
-
Notifications
You must be signed in to change notification settings - Fork 974
fix(locale): ko modified street_name to street_name_part #3485
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
✅ Deploy Preview for fakerjs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #3485 +/- ##
=======================================
Coverage 99.97% 99.97%
=======================================
Files 2880 2880
Lines 220417 220507 +90
Branches 949 949
=======================================
+ Hits 220362 220454 +92
+ Misses 55 53 -2
🚀 New features to boost your workflow:
|
Could you split the state name changes to a separate PR? Keeps it cleaner. |
f2b68b4
to
d0461cd
Compare
Yes, I've split the state name changes to this PR: #3487 |
Fixed street pattern usage for ko locale #2062
street_name.ts
modified tostreet_name_part.ts
.street_pattern.ts
correspondingly modified.Additionally:
Updated data to more accurately represent state names and street names.
Street names:
Added data to
street_name_part.ts
(formerlystreet_name.ts
).Added and modified data in
street_suffix.ts
. The previous suffixes instreet_suffix.ts
were not suffixes for street names, but rather for zones/areas. Modified the suffixes for accuracy, and added additional suffixes involving numbers as common for Korean street names.State names:
Changes to state names have been moved to PR #3487