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

Support for deep merge #20

Open
davegraham92120 opened this issue Aug 4, 2018 · 1 comment
Open

Support for deep merge #20

davegraham92120 opened this issue Aug 4, 2018 · 1 comment
Assignees
@toddgeist @davegraham92120 @lancebrandenberg
Labels
enhancement help wanted

Comments

@davegraham92120
Copy link
Contributor

Here's a modified JSON.Merge function that performs a deep merge. There's a new test, and all tests are passing. Can you please have a look and help make sure it's solid?

JSONAdditions-GH 20.fmp12.zip
@davegraham92120
Copy link
Contributor Author

I found and fixed an issue where a deep merge would fail if the sourceValue is an object and there was no target object. See the revised file with the fix and new test. All tests are passing.
JSONAdditions-GH 20v2.fmp12.zip

@DanShockley DanShockley mentioned this issue Apr 21, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@toddgeist toddgeist

@davegraham92120 davegraham92120

@lancebrandenberg lancebrandenberg

Labels
enhancement help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@toddgeist @davegraham92120 @lancebrandenberg