-
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
The examples for Faker::Date.between do not work #2056
Comments
Zeragamba
changed the title
Issue title goes here
The examples for Faker::Date.between do not work
Jun 12, 2020
The methods (.days ago and alike) are part of |
The documents need to be more explicit in my opinion. Either explaining the
dependency or with a more appropriate example.
…On Sun, Jun 14, 2020 at 3:18 AM Daniel Tiringer ***@***.***> wrote:
The methods (.days ago and alike) are part of activesupport. Requiring
active_support/time makes the example above work.
Should this be mentioned in the documentation (YARD and /docs) as a
solution, or should we investigate alternative solutions?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#2056 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIBHEBK5AZALT4FLK4RPC3RWR2VFANCNFSM4N4UJJLA>
.
|
I think it would be good to update the examples to show both a Rails and Rails-less. |
The issue has been resolved by #2061. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
The examples for
Faker::Date.between(from: 2.days.ago, to: Date.today)
andFaker::Date.between_except(from: 1.year.ago, to: 1.year.from_now, excepted: Date.today)
do not work.To Reproduce
Expected behavior
I should see a date between the given arguments passed to
between
Additional context
My guess is that these examples require
activesupport
in order to get the extensions on the integer class.The text was updated successfully, but these errors were encountered: