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

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
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