-
Notifications
You must be signed in to change notification settings - Fork 19
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
Flatten all the editing methods into DAG
#34
Conversation
37cb13b
to
3ca0d35
Compare
@Kixunil, what do you think of this? I think most of the stuff has been pretty good and feels much better, except the things I've flagged up - this still took me most of yesterday, though 😆. |
12c3b88
to
48a937e
Compare
04fa3bc
to
fdac6ea
Compare
Right I think in the interests of letting other stuff happen and not generate merge conflicts, I think we should merge this. I fixed a very rogue crash bug that had been introduced earlier, but it's seeming pretty stable now. We could do with some integration tests, but that's for another PR I reckon. @Kixunil, what do you think? |
fdac6ea
to
393df72
Compare
Sounds good. We can always refactor more later. Doing it in chunks is better anyway. |
393df72
to
1609ffd
Compare
Coool! Done another rebase to fix merge conflicts, now merging... |
After #34, inserting into JSON objects is now possible. This is now reflected in the README.
Resolves #24.
TODO:
CannotInsertInto
andCannotReplace
with something likeInvalidChild