Skip to content
This repository has been archived by the owner on Oct 15, 2022. It is now read-only.

DaysBetween - Added a few more triggers #1230

Merged
merged 1 commit into from
Jul 6, 2015
Merged

DaysBetween - Added a few more triggers #1230

merged 1 commit into from
Jul 6, 2015

Conversation

Bujiraso
Copy link
Contributor

@Bujiraso Bujiraso commented Jul 4, 2015

I have completed one of the five minute fixes suggested by the mailer, specifically issue #1221 . This is my first contribution/pull request so please let me know if there's anything more I should do.

@moollaza
Copy link
Member

moollaza commented Jul 6, 2015

@Bujiraso Thanks for taking the time to contribute! We really appreciate it. This LGTM 👍

Feel free to stick around and comment/help on other PR's & Issues or even submit more Instant Answers!

@moollaza moollaza self-assigned this Jul 6, 2015
@@ -90,6 +90,30 @@ ddg_goodie_test(
result => 45
},
),
'number of days between jan 1 and 15th feb' => test_zci(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

moollaza added a commit that referenced this pull request Jul 6, 2015
DaysBetween - Added a few more triggers
@moollaza moollaza merged commit 1400d09 into duckduckgo:master Jul 6, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants