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

More DRY, uses splatting, added help, changed output type, updated README #2

Merged
merged 1 commit into from Jan 30, 2017

Conversation

briantist
Copy link

I know these are some major changes, but I think they're useful.

The big breaking change is changing the output type to a [TimeSpan]. I think this is important; it's more versatile and contains more information. One can easily get the days out of the timespan returned, or use it directly for something that needs a timespan, but one cannot get the missing information if a number of days is returned.

…ange output type to TimeSpan, update README
@itadder
Copy link
Owner

itadder commented Jan 30, 2017

Thanks for the update. and for adding help.
Briantist made some changes to my simple script, and updated the readme.

Thanks Briantist

@itadder itadder merged commit eebed0b into itadder:master Jan 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants