{"action":"created","issue":{"url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/20","repository_url":"https://api.github.com/repos/KyleAMathews/deepmerge","labels_url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/20/labels{/name}","comments_url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/20/comments","events_url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/20/events","html_url":"https://github.com/KyleAMathews/deepmerge/pull/20","id":81154014,"number":20,"title":"Added capability to specify whether arrays should be merged or replaced","user":{"login":"nileshchavan","id":7615328,"avatar_url":"https://avatars.githubusercontent.com/u/7615328?v=3","gravatar_id":"","url":"https://api.github.com/users/nileshchavan","html_url":"https://github.com/nileshchavan","followers_url":"https://api.github.com/users/nileshchavan/followers","following_url":"https://api.github.com/users/nileshchavan/following{/other_user}","gists_url":"https://api.github.com/users/nileshchavan/gists{/gist_id}","starred_url":"https://api.github.com/users/nileshchavan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nileshchavan/subscriptions","organizations_url":"https://api.github.com/users/nileshchavan/orgs","repos_url":"https://api.github.com/users/nileshchavan/repos","events_url":"https://api.github.com/users/nileshchavan/events{/privacy}","received_events_url":"https://api.github.com/users/nileshchavan/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/KyleAMathews/deepmerge/labels/waiting%20for%20response","name":"waiting for response","color":"01FF70"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2015-05-26T21:55:24Z","updated_at":"2016-09-27T18:48:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/KyleAMathews/deepmerge/pulls/20","html_url":"https://github.com/KyleAMathews/deepmerge/pull/20","diff_url":"https://github.com/KyleAMathews/deepmerge/pull/20.diff","patch_url":"https://github.com/KyleAMathews/deepmerge/pull/20.patch"},"body":"This change is inspired by https://forum.jquery.com/topic/jquery-extend-modifies-but-not-replaces-array-properties-on-deep-copy\r\n\r\nI ran into the exact same issue mentioned in the above link. I thought it would be useful to provide an option to user to choose whether arrays should be merged or replaced. Let me know what you think about it."},"comment":{"url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/comments/249960270","html_url":"https://github.com/KyleAMathews/deepmerge/pull/20#issuecomment-249960270","issue_url":"https://api.github.com/repos/KyleAMathews/deepmerge/issues/20","id":249960270,"user":{"login":"nileshchavan","id":7615328,"avatar_url":"https://avatars.githubusercontent.com/u/7615328?v=3","gravatar_id":"","url":"https://api.github.com/users/nileshchavan","html_url":"https://github.com/nileshchavan","followers_url":"https://api.github.com/users/nileshchavan/followers","following_url":"https://api.github.com/users/nileshchavan/following{/other_user}","gists_url":"https://api.github.com/users/nileshchavan/gists{/gist_id}","starred_url":"https://api.github.com/users/nileshchavan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nileshchavan/subscriptions","organizations_url":"https://api.github.com/users/nileshchavan/orgs","repos_url":"https://api.github.com/users/nileshchavan/repos","events_url":"https://api.github.com/users/nileshchavan/events{/privacy}","received_events_url":"https://api.github.com/users/nileshchavan/received_events","type":"User","site_admin":false},"created_at":"2016-09-27T18:48:36Z","updated_at":"2016-09-27T18:48:36Z","body":"@TehShrike, I haven't checked other pull requests but your solution looks good to me. I think delegating array merging is a good idea."}}
{"actor":{"display_login":"nileshchavan"}}