Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Estonian and Latvian locales #1140

Merged

Conversation

maciej-szlosarczyk
Copy link
Contributor

Currently supports:

  • Regions
  • Cities
  • Names
  • Phone numbers
  • Cellphone numbers

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.

Could you please rebase with master and fix the rubocop violations?

@vbrazo
Copy link
Member

vbrazo commented May 18, 2018

Could you also update the changelog.md? You should add this PR and your GitHub ID.

@vbrazo vbrazo force-pushed the master branch 3 times, most recently from 6394a28 to d11bd7e Compare May 20, 2018 20:41
@maciej-szlosarczyk maciej-szlosarczyk force-pushed the add-estonia-and-latvia-locales branch 3 times, most recently from ed95132 to a6c49b0 Compare May 21, 2018 14:23
@maciej-szlosarczyk maciej-szlosarczyk force-pushed the add-estonia-and-latvia-locales branch from a6c49b0 to 4596679 Compare May 21, 2018 16:14
@maciej-szlosarczyk maciej-szlosarczyk force-pushed the add-estonia-and-latvia-locales branch from 4596679 to 8abeeb0 Compare May 21, 2018 16:19
* Regions
* Cities
* Names
* Phone numbers
@maciej-szlosarczyk maciej-szlosarczyk force-pushed the add-estonia-and-latvia-locales branch from 8abeeb0 to ee1ea90 Compare May 21, 2018 16:25
@maciej-szlosarczyk
Copy link
Contributor Author

Hey @vbrazo, now it should be done.

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.

Well, overall looks good. I've just added a few other tests and now I believe it's good to go.
I saw that you're a first time-contributor. Welcome 🥇 Do not forget to read our contributing doc.
Pretty awesome locales. Let me know if you want to contribute in other modules.

@vbrazo vbrazo merged commit a526091 into faker-ruby:master May 27, 2018
@vbrazo vbrazo self-requested a review July 19, 2018 01:31
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* Add missing lorem locale and update changelog

* Add Estonian and Latvian fake data:

* Regions
* Cities
* Names
* Phone numbers

* Add more locale tests
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants