-
Notifications
You must be signed in to change notification settings - Fork 18
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
round-trip error with ###NaN #11
Comments
I think there are two issues, and I think they're really two manifestations with the same issue.
|
Glad I put my research into writing, but I don't think this should be a priority for the moment. |
Seems that CircleCI link no longer works for me... should've taken a screenshot. Adding a section to the README with "known limitations" would perhaps be the best short-term action here. |
I think this makes sense to leave something open as a documentation fix. Fixing the underlying problem probably won't be a priority, so this can be closed as "PR welcome" once the documentation is done. |
https://circleci.com/gh/lambdaisland/deep-diff/65?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
Something the generative tests caught, not sure this is a big issue but something to look into.
The text was updated successfully, but these errors were encountered: