Skip to content
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

fix-chinese-city #1591

Merged
merged 2 commits into from
Apr 2, 2019
Merged

fix-chinese-city #1591

merged 2 commits into from
Apr 2, 2019

Conversation

locez
Copy link
Contributor

@locez locez commented Apr 1, 2019

Replace old data with real Chinese major city names
When I need to generate a city that can be marked on the map, the old data is not available because they are not real, I think these real city names will be more useful.

Replace old data with real Chinese major city names
When I need to generate city locations that can be marked on the map, the old data is not available because they are not real
Fix space in city array
Copy link
Member

@vbrazo vbrazo left a comment

Choose a reason for hiding this comment

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

thanks 👍

@vbrazo vbrazo merged commit abf8250 into faker-ruby:master Apr 2, 2019
@locez locez deleted the fix-chinese-city branch April 3, 2019 06:44
michebble pushed a commit to michebble/faker that referenced this pull request Feb 16, 2020
* fix-chinese-city

Replace old data with real Chinese major city names
When I need to generate city locations that can be marked on the map, the old data is not available because they are not real

* Fix space in city array

Fix space in city array
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* fix-chinese-city

Replace old data with real Chinese major city names
When I need to generate city locations that can be marked on the map, the old data is not available because they are not real

* Fix space in city array

Fix space in city array
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants