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

Adds turkish phone number formats and test. #1106

Merged
merged 3 commits into from
May 22, 2018

Conversation

ZASMan
Copy link
Contributor

@ZASMan ZASMan commented Jan 3, 2018

Adds some formatting for turkish phone numbers with 90 country code and a test.

@ZASMan
Copy link
Contributor Author

ZASMan commented Apr 20, 2018

Can I get a review on this @stympy

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.

Yes, you can get a review on this PR :)

Could you please rebase with master, fix the Rubocop violations and update the changelog.md?

@vbrazo vbrazo force-pushed the master branch 3 times, most recently from 6394a28 to d11bd7e Compare May 20, 2018 20:41
Zane Sandin and others added 2 commits May 21, 2018 21:17
Adds changelog and fixes violations

Adds turkish phone number formats and a test.

Adds changelog and fixes violations
@ZASMan ZASMan force-pushed the add_turkish_phone_formats branch from 5695852 to 36edc5f Compare May 22, 2018 01:17
@ZASMan
Copy link
Contributor Author

ZASMan commented May 22, 2018

yes sir, I think I have it now!

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.

Awesome. Thanks for contributing 👍

@vbrazo vbrazo merged commit ea829c5 into faker-ruby:master May 22, 2018
@@ -33,6 +33,10 @@ def test_tr_address_fields
assert Faker::Address.country.is_a? String
end

def test_tr_phone_number_fields
assert Faker::PhoneNumber.phone_number.is_a? String
end
Copy link
Member

Choose a reason for hiding this comment

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

👍

@vbrazo vbrazo self-requested a review July 19, 2018 01:32
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* Adds turkish phone number formats and a test.

Adds changelog and fixes violations

Adds turkish phone number formats and a test.

Adds changelog and fixes violations

* Updates changelog
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