-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Adds turkish phone number formats and test. #1106
Conversation
Can I get a review on this @stympy |
There was a problem hiding this 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
?
6394a28
to
d11bd7e
Compare
Adds changelog and fixes violations Adds turkish phone number formats and a test. Adds changelog and fixes violations
5695852
to
36edc5f
Compare
yes sir, I think I have it now! |
There was a problem hiding this 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 👍
@@ -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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
* 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
Adds some formatting for turkish phone numbers with
90
country code and a test.