-
Notifications
You must be signed in to change notification settings - Fork 5
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
Not optimal number of diffs #3
Comments
This jsondiffpatch library seems to handle that case correctly (example) I haven't been able to look and see what the implementation difference is but maybe we can learn from it and update the arraydiff implementation. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I suppose this happens because array has repeating elements.
Here is test case: https://github.com/vmakhaev/arraydiff-testcase
As you see, I remove one item and expect only one
RemoveDiff
, but get:The text was updated successfully, but these errors were encountered: