Skip to content

Allow bumped version of aniso8601 #1042

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

Closed
wants to merge 1 commit into from

Conversation

jasonrfarkas
Copy link

In version 6 aniso8601 some rounding bugs were introduced.
See the issue reported here: https://bitbucket.org/nielsenb/aniso8601/issues/24/float-induced-rounding-errors-when-parsing
See the commit that fixed it here: https://bitbucket.org/nielsenb/aniso8601/commits/b4aca8ffc25588128d1c463ca735c8d6f180f55a
Allow for a higher version to allow graphene users to avoid these issues.

In version 6 aniso8601 some rounding bugs were introduced. 
See the issue reported here: https://bitbucket.org/nielsenb/aniso8601/issues/24/float-induced-rounding-errors-when-parsing
See the commit that fixed it here: https://bitbucket.org/nielsenb/aniso8601/commits/b4aca8ffc25588128d1c463ca735c8d6f180f55a
Allow for a higher version to allow graphene users to avoid these issues.
@NyanKiyoshi
Copy link
Contributor

There is #1009 already opened for it 👍

@jasonrfarkas
Copy link
Author

Ah great! - this one doesn't have the merge conflict though :)

@jkimbo
Copy link
Member

jkimbo commented Jul 24, 2019

Thanks for the PR @jasonrfarkas but I just merged #1009 so I'm going to close this.

@jkimbo jkimbo closed this Jul 24, 2019
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