Skip to content

fix(locale): es cities should be in city_name not city_prefix, remove extraneous city_suffix#2109

Merged
ST-DDT merged 1 commit into
faker-js:nextfrom
matthewmayer:chore/es-city-names
Apr 29, 2023
Merged

fix(locale): es cities should be in city_name not city_prefix, remove extraneous city_suffix#2109
ST-DDT merged 1 commit into
faker-js:nextfrom
matthewmayer:chore/es-city-names

Conversation

@matthewmayer

@matthewmayer matthewmayer commented Apr 29, 2023

Copy link
Copy Markdown
Contributor

Both es and es_MX have a real list of cities incorrectly in city_prefix
instead of city_name.

Also removed unused English suffixes in es_MX locale

@matthewmayer matthewmayer requested a review from a team as a code owner April 29, 2023 04:39
@matthewmayer matthewmayer self-assigned this Apr 29, 2023
@matthewmayer matthewmayer added p: 1-normal Nothing urgent c: locale Permutes locale definitions m: location Something is referring to the location module labels Apr 29, 2023
@codecov

codecov Bot commented Apr 29, 2023

Copy link
Copy Markdown

Codecov Report

Merging #2109 (b8ada1f) into next (3cfeb38) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             next    #2109      +/-   ##
==========================================
- Coverage   99.59%   99.59%   -0.01%     
==========================================
  Files        2567     2566       -1     
  Lines      243371   243349      -22     
  Branches     1252     1252              
==========================================
- Hits       242396   242363      -33     
- Misses        948      959      +11     
  Partials       27       27              
Impacted Files Coverage Δ
src/locales/es/location/city_name.ts 100.00% <ø> (ø)
src/locales/es_MX/location/city_name.ts 100.00% <ø> (ø)
src/locales/es/location/city_pattern.ts 100.00% <100.00%> (ø)
src/locales/es/location/index.ts 100.00% <100.00%> (ø)
src/locales/es_MX/location/city_pattern.ts 100.00% <100.00%> (ø)
src/locales/es_MX/location/index.ts 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

@matthewmayer matthewmayer changed the title fix(locale): es cities should be in city_name not city_suffix, remove extraneous city_suffix fix(locale): es cities should be in city_name not city_prefix, remove extraneous city_suffix Apr 29, 2023
@ST-DDT ST-DDT requested review from a team April 29, 2023 08:11
@ST-DDT ST-DDT merged commit 789c8a2 into faker-js:next Apr 29, 2023
matthewmayer added a commit to matthewmayer/faker that referenced this pull request Apr 30, 2023
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

4 participants