swap missing parent assignment of {} to after last target check to av…

[dkebler]

Here it be. I added three tests to that use a setter and observable and it passes all of those and your tests as long as the that parent assignement is after the break. Just swap those back and you will fail the test

BTW there was no issue with the merge option. (included a test for that).

---

Also I think you should include documentation readme for the currently undocumented options, (i.e. merge/function,split/separator)

FYI I have a bit more robust handler for variations of the path format. Handles also multiple delimiters. I just call before calling set it might be nice to incorporate directly to avoid unnecessary joining and splitting. Let me know if that is something you want to include and I'll make a pr for that.

[jonschlinkert]

Hi @dkebler, thanks for the PR and sorry for the late response. I refactored this library and will be doing a major release shortly, but I'm going to merge this anyway to acknowledge the time and effort you took to contribute.

I'll also make sure to include your suggestions in the release.

Thank you.

[jonschlinkert]

FYI I have a bit more robust handler for variations of the path format. Handles also multiple delimiters. I just call before calling set it might be nice to incorporate directly to avoid unnecessary joining and splitting. Let me know if that is something you want to include and I'll make a pr for that.

forgot to comment on this.. if you're still interested, I think it would be easier to discuss over a PR if you feel like doing one. thanks!

[dkebler]

glad you did that. I've been forced to use my fork.

As to improved handler It's been awhile but I'll refesh the memory cells make a new PR.