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 Faker::SouthPark #1314

Merged
merged 4 commits into from
Jul 17, 2018
Merged

Add Faker::SouthPark #1314

merged 4 commits into from
Jul 17, 2018

Conversation

saurabhudaniya200
Copy link
Contributor

Description

Feature request for South Park consisting 55 characters and 17 quotes

Tests

1092 tests, 79817 assertions, 0 failures, 0 errors, 0 pendings, 1 omissions, 0 notifications
100% passed

Rubocop

358 files inspected, no offenses detected

@@ -0,0 +1,15 @@
# frozen_string_literal: true
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Magic comment added by Rubocop

Copy link
Member

Choose a reason for hiding this comment

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

#1311 may explain the magic comments.

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.

Overall looks good.

Could you also update the README.md?

@saurabhudaniya200
Copy link
Contributor Author

Sure @vbrazo, will update the README.md

@saurabhudaniya200
Copy link
Contributor Author

@vbrazo Done, Updated README.md to reflect the changes

@vbrazo vbrazo merged commit 3de9c6c into faker-ruby:master Jul 17, 2018
@saurabhudaniya200 saurabhudaniya200 deleted the south_park branch July 23, 2018 10:34
@saurabhudaniya200 saurabhudaniya200 restored the south_park branch July 23, 2018 10:48
@vbrazo
Copy link
Member

vbrazo commented Oct 12, 2018

@saurabhudaniya200 just a heads up:

I'm adding a namespace for your SouthPark in this PR #1403. Since we haven't released this object yet, we don't need to deprecate it. Thanks 👍

davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* Added south park feature

* Renamed south_park.doc to south_park.md

* Updated README to reflect newly added SouthPark feature

* Update CHANGELOG.md
# 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