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 output functionality in ISO 8601 format #38

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

Add output functionality in ISO 8601 format #38

wants to merge 20 commits into from

Conversation

ghost
Copy link

@ghost ghost commented May 11, 2015

Fixes #33 - just use :format => :iso.

I had to refactor the result map loop a bit for the special T operator in the middle of the output when an hour, minute, or second exists, but otherwise most of the other code is unaffected. I've also updated the documentation and added results to the spec file. There are some glitches that I'd like to fix if I had more free time (e.g. 0 input produces an output of P), but this is enough to satisfy my needs. I also didn't program the parsing portion so it will fail some checks.

I'm submitting this PR as a courtesy with a new version tag (v0.10.7), but please let me know if you'd like to see any improvements before including it in your code.

Cheers!

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.

ISO 8601 Durations support?
1 participant