{"action":"created","issue":{"url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/43","repository_url":"https://api.github.com/repos/KyleAMathews/deepmerge","labels_url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/43/labels{/name}","comments_url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/43/comments","events_url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/43/events","html_url":"https://github.com/KyleAMathews/deepmerge/pull/43","id":180300171,"number":43,"title":"Added failing tests case for {a:[]} {a:undefined}","user":{"login":"callumacrae","id":472830,"avatar_url":"https://avatars.githubusercontent.com/u/472830?v=3","gravatar_id":"","url":"https://api.github.com/users/callumacrae","html_url":"https://github.com/callumacrae","followers_url":"https://api.github.com/users/callumacrae/followers","following_url":"https://api.github.com/users/callumacrae/following{/other_user}","gists_url":"https://api.github.com/users/callumacrae/gists{/gist_id}","starred_url":"https://api.github.com/users/callumacrae/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/callumacrae/subscriptions","organizations_url":"https://api.github.com/users/callumacrae/orgs","repos_url":"https://api.github.com/users/callumacrae/repos","events_url":"https://api.github.com/users/callumacrae/events{/privacy}","received_events_url":"https://api.github.com/users/callumacrae/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-09-30T13:10:43Z","updated_at":"2016-09-30T14:33:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/KyleAMathews/deepmerge/pulls/43","html_url":"https://github.com/KyleAMathews/deepmerge/pull/43","diff_url":"https://github.com/KyleAMathews/deepmerge/pull/43.diff","patch_url":"https://github.com/KyleAMathews/deepmerge/pull/43.patch"},"body":"Hello! Currently in the process of moving away from lodash, looking to use this library for deep merging.\r\n\r\nCurrently, if you merge `{ array: [] }` and `{ array: undefined }`, you get `{ array: null }`. This seems… odd.\r\n\r\nMy preferred behaviour would be for the result to be `{ array: [] }` - this is how `lodash/merge` behaves - but at the very least it should probably by `{ array: undefined }`.\r\n\r\nThoughts? Happy to add a fix to this PR, whatever is decided."},"comment":{"url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/comments/250760143","html_url":"https://github.com/KyleAMathews/deepmerge/pull/43#issuecomment-250760143","issue_url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/43","id":250760143,"user":{"login":"TehShrike","id":1141869,"avatar_url":"https://avatars.githubusercontent.com/u/1141869?v=3","gravatar_id":"","url":"https://api.github.com/users/TehShrike","html_url":"https://github.com/TehShrike","followers_url":"https://api.github.com/users/TehShrike/followers","following_url":"https://api.github.com/users/TehShrike/following{/other_user}","gists_url":"https://api.github.com/users/TehShrike/gists{/gist_id}","starred_url":"https://api.github.com/users/TehShrike/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TehShrike/subscriptions","organizations_url":"https://api.github.com/users/TehShrike/orgs","repos_url":"https://api.github.com/users/TehShrike/repos","events_url":"https://api.github.com/users/TehShrike/events{/privacy}","received_events_url":"https://api.github.com/users/TehShrike/received_events","type":"User","site_admin":false},"created_at":"2016-09-30T14:33:45Z","updated_at":"2016-09-30T14:33:45Z","body":"I can dig that.\r\n\r\nWe should have tests making sure that `undefined` will properly overwrite `null` and a primitive value like a number, too, and then I'd be cool with that solution."}}
{"actor":{"display_login":"TehShrike"}}