{"action":"created","issue":{"url":"https://api.github.com/repos/technomancy/leiningen/issues/1981","labels_url":"https://api.github.com/repos/technomancy/leiningen/issues/1981/labels{/name}","comments_url":"https://api.github.com/repos/technomancy/leiningen/issues/1981/comments","events_url":"https://api.github.com/repos/technomancy/leiningen/issues/1981/events","html_url":"https://github.com/technomancy/leiningen/pull/1981","id":105308179,"number":1981,"title":"Remove metadata on updated key with update-in","user":{"login":"j-allard","id":943725,"avatar_url":"https://avatars.githubusercontent.com/u/943725?v=3","gravatar_id":"","url":"https://api.github.com/users/j-allard","html_url":"https://github.com/j-allard","followers_url":"https://api.github.com/users/j-allard/followers","following_url":"https://api.github.com/users/j-allard/following{/other_user}","gists_url":"https://api.github.com/users/j-allard/gists{/gist_id}","starred_url":"https://api.github.com/users/j-allard/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/j-allard/subscriptions","organizations_url":"https://api.github.com/users/j-allard/orgs","repos_url":"https://api.github.com/users/j-allard/repos","events_url":"https://api.github.com/users/j-allard/events{/privacy}","received_events_url":"https://api.github.com/users/j-allard/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-09-08T06:02:48Z","updated_at":"2015-11-21T16:09:30Z","closed_at":"2015-11-21T16:09:30Z","pull_request":{"url":"https://api.github.com/repos/technomancy/leiningen/pulls/1981","html_url":"https://github.com/technomancy/leiningen/pull/1981","diff_url":"https://github.com/technomancy/leiningen/pull/1981.diff","patch_url":"https://github.com/technomancy/leiningen/pull/1981.patch"},"body":"Okay so here my second take on this.\r\nSorry for the \"duplicate\" PR, I couldn't reopen the other one because I forced-pushed to my branch :(\r\n\r\nFIxing this https://github.com/technomancy/leiningen/issues/1969\r\nContinuity of this https://github.com/technomancy/leiningen/pull/1977\r\n\r\nSo basically this is removing the existent metadata on keys you update with the update-in task and this makes the rest of the merge process to behave as expected (i.e the keys updated using update-in have priority).\r\n\r\nAny thoughts or something obvious I might have missed ?"},"comment":{"url":"https://api.github.com/repos/technomancy/leiningen/issues/comments/158659168","html_url":"https://github.com/technomancy/leiningen/pull/1981#issuecomment-158659168","issue_url":"https://api.github.com/repos/technomancy/leiningen/issues/1981","id":158659168,"user":{"login":"hyPiRion","id":504876,"avatar_url":"https://avatars.githubusercontent.com/u/504876?v=3","gravatar_id":"","url":"https://api.github.com/users/hyPiRion","html_url":"https://github.com/hyPiRion","followers_url":"https://api.github.com/users/hyPiRion/followers","following_url":"https://api.github.com/users/hyPiRion/following{/other_user}","gists_url":"https://api.github.com/users/hyPiRion/gists{/gist_id}","starred_url":"https://api.github.com/users/hyPiRion/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hyPiRion/subscriptions","organizations_url":"https://api.github.com/users/hyPiRion/orgs","repos_url":"https://api.github.com/users/hyPiRion/repos","events_url":"https://api.github.com/users/hyPiRion/events{/privacy}","received_events_url":"https://api.github.com/users/hyPiRion/received_events","type":"User","site_admin":false},"created_at":"2015-11-21T16:09:30Z","updated_at":"2015-11-21T16:09:30Z","body":"Unfortunately, this doesn't solve all cases of this problem: If you only drop metadata, then merging other profiles in would merge \"as usual\". E.g. if you do `lein update-in :jvm-opts concat [\"-Xdebug\"] -- repl`, then if the :repl profile contains `:jvm-opts ^:replace []` you'll end up with an empty vector for :jvm-opts. The question, of course, is whether this is \"as expected\" or not – I've explained this issue in #1969.\r\n\r\nI think I want to defer fixing #1969 until 3.0.0 for now, unless a more immediate use case appears."}}