-
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 Faker::Date.in_date_period #1755
Add Faker::Date.in_date_period #1755
Conversation
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.
Just some minor feedback :)
Co-Authored-By: Connor Shea <connor.james.shea@gmail.com>
Co-Authored-By: Connor Shea <connor.james.shea@gmail.com>
LGTM 👍 |
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.
Thanks for this! Looks good to me!
a2a9f17
to
4384168
Compare
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.
Thanks 👍
* Add new class method in_date_period to Faker::Date * Add unit test cases for Faker::Date.in_date_period class method * Add YARD documentation for Faker::Date.in_date_period class method * Add more documentation for Faker::Date.in_date_period class method * Fix method version to be next instead of 2.3.0 * Fix documentation description Co-Authored-By: Connor Shea <connor.james.shea@gmail.com> * Fix documentation description Co-Authored-By: Connor Shea <connor.james.shea@gmail.com> * Update the documentation examples * revert commit b3ba644 for updating docs * Remove extra lines
* Add new class method in_date_period to Faker::Date * Add unit test cases for Faker::Date.in_date_period class method * Add YARD documentation for Faker::Date.in_date_period class method * Add more documentation for Faker::Date.in_date_period class method * Fix method version to be next instead of 2.3.0 * Fix documentation description Co-Authored-By: Connor Shea <connor.james.shea@gmail.com> * Fix documentation description Co-Authored-By: Connor Shea <connor.james.shea@gmail.com> * Update the documentation examples * revert commit b3ba644 for updating docs * Remove extra lines
Fixes #1746