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

Support parsing microseconds with a parse_exact/1 function #16

Closed
wants to merge 2 commits into from
Closed

Support parsing microseconds with a parse_exact/1 function #16

wants to merge 2 commits into from

Conversation

aseigo
Copy link
Contributor

@aseigo aseigo commented Mar 17, 2015

Tried to make this as non-invasive and backwards compatible as possible. I need the ability to parse exact dates out for later reformating ... this makes it trivial to go from string containing an iso8601 datetime with tz to a string in UTC:

1> iso8601:format(iso8601:parse_exact("2015-02-23T14:59:49.717+01:00")).
<<"2015-02-23T13:59:49.717Z">>

Am happy to make adjustment as needed.

@oubiwann
Copy link
Member

@aseigo This looks nice. We'll need to add some unit tests, though.

Could you rebase and re-push?

Note that the original project has a new maintainer and a new home:

That being said, the old links to the original repo will redirect to the new home -- both in the browser and via the cli.

@hachreak
Copy link
Contributor

hi @aseigo and @oubiwann , what about this PR? :)
Any news? Can I contribute? :D

@oubiwann
Copy link
Member

@hachreak Sure! This PR's branch has conflicts -- if you want to take it over (preserving the history/work done so far) and get those resolved, that would be great.

It also needs some unit tests around the new feature (and tests that ensure pre-existing functionality is not broken in anyway).

@aseigo
Copy link
Contributor Author

aseigo commented Sep 22, 2016

Obsoleted by pr #31

@aseigo aseigo closed this Sep 22, 2016
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.

3 participants