Skip to content
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

Merged
merged 10 commits into from
Oct 10, 2019

Conversation

AmrAdelKhalil
Copy link
Contributor

Fixes #1746

lib/faker/default/date.rb Outdated Show resolved Hide resolved
Copy link
Member

@connorshea connorshea left a 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 :)

lib/faker/default/date.rb Outdated Show resolved Hide resolved
lib/faker/default/date.rb Outdated Show resolved Hide resolved
doc/default/date.md Outdated Show resolved Hide resolved
doc/default/date.md Show resolved Hide resolved
AmrAdelKhalil and others added 4 commits September 20, 2019 01:32
Co-Authored-By: Connor Shea <connor.james.shea@gmail.com>
Co-Authored-By: Connor Shea <connor.james.shea@gmail.com>
lib/faker/default/date.rb Outdated Show resolved Hide resolved
@connorshea
Copy link
Member

LGTM 👍

@vbrazo vbrazo changed the title Generate date by year Add Faker::Date.in_date_period Sep 23, 2019
Copy link
Member

@bpleslie bpleslie left a 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!

@vbrazo vbrazo force-pushed the master branch 3 times, most recently from a2a9f17 to 4384168 Compare September 30, 2019 07:10
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.

Thanks 👍

@vbrazo vbrazo merged commit 69ecdde into faker-ruby:master Oct 10, 2019
vbrazo added a commit that referenced this pull request Oct 10, 2019
michebble pushed a commit to michebble/faker that referenced this pull request Feb 16, 2020
* 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
michebble pushed a commit to michebble/faker that referenced this pull request Feb 16, 2020
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* 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
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a method for generating a date in a given year to Faker::Date
5 participants