-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
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.