search

cujojs / jiff

JSON Patch and diff based on rfc6902
Other
627 stars 41 forks source link

Remove patchInPlace? #3

Open briancavalier opened 10 years ago

briancavalier commented 10 years ago

Now that patching is atomic by default, we may just want to ditch patchInPlace if patching is "fast enough". It's pretty hazardous, especially now that test is implemented.

danieljuhl commented 8 years ago

When working on pure data, I get the idea, but if you work on eg. a Mongoose document, it could still make sense to do a patchInPlace