-
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
Add bible entries #1656
Add bible entries #1656
Conversation
Hi,I submitted a pull request and it was assigned.I was just wondering why it has taken this long to be merged considering that all test passed.
thanks.
Enow B. Mbi,
On Friday, July 12, 2019, 8:08:52 AM GMT+1, Vitor Oliveira <notifications@github.com> wrote:
Assigned #1656 to @enowmbi.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
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.
Changes look good to me. Thanks for the PR
a2a9f17
to
4384168
Compare
flexible :bible | ||
|
||
class << self | ||
def character |
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.
Missing Yard docs for these new methods. Check CONTRIBUTING.md for details
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.
@Zeragamba, can you verify. 🙏🏾
Added bible entries allowing users to use quotations, locations, and characters from the bible. Entries are in the religion category. e.g.
Faker::Religion::Bible.location #=> 'Jerusalem'
Faker::Religion::Bible.character #=> 'Jesus'
Faker::Religion::Bible.quote #=> 'Seek ye first the kingdom of God'