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.

Sign up for GitHub

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

Jump to bottom

Add param validation to DateTime::Duration->multiply #73

Closed

danstewart wants to merge 1 commit into houseabsolute:master from danstewart:integer_multiplication

danstewart commented Feb 18, 2018

For #25
It only took me a year.

Wasn't sure the best way to handle the additional overload arguments to multiply, so just set slurpy => 1.


          Add param validation to DateTime::Duration->multiply

2ae60af

preaction approved these changes

View reviewed changes

Member

autarch commented Mar 24, 2018

Merged from the CLI. Thanks!

autarch closed this

danstewart deleted the integer_multiplication branch

March 24, 2018 20:49

abraxxa mentioned this pull request

DateTime::Duration shouldn't allow non-integer multiplication #25

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment