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.

Sign up for GitHub

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

Jump to bottom

make backtrack iterative should fix #223 #237

Merged
merged 2 commits into from
Jan 1, 2020
Merged

make backtrack iterative should fix #223 #237

merged 2 commits into from
Jan 1, 2020

Conversation

aryelu
Copy link
Contributor

@aryelu aryelu commented Jun 28, 2018 •
edited
Loading

@ofiratt thanks for the help

@aryelu
Copy link
Contributor Author

aryelu commented Jun 28, 2018 •
edited
Loading

should fix #223 #156

@ipassynk
Copy link

Any idea if this PR is going to be merged? We are running into Maximum stack trace issue with big arrays. Thanks

@jpillora
Copy link

jpillora commented Oct 2, 2019 •
edited
Loading

published this PR to npm as jsondiffpatch237@0.3.11-alpha

@benjamine
Copy link
Owner

thanks @aryelu , sorry for the delay, I want to add a bit more test coverage to LCS, but this looks good,
I'll merge and add a few more test cases later 👍

@benjamine benjamine merged commit b0f0566 into benjamine:master Jan 1, 2020
benjamine pushed a commit that referenced this pull request Jan 2, 2020
@beneidel
v0.4
b0a41f9 
- dropping support for node < 8
- multiple typescript annotation updates
- LCS non-recursive implementation  (see #237 )
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aryelu @ipassynk @jpillora @benjamine