{"action":"created","issue":{"url":"https://api.github.com/repos/CocoaPods/CocoaPods/issues/3548","labels_url":"https://api.github.com/repos/CocoaPods/CocoaPods/issues/3548/labels{/name}","comments_url":"https://api.github.com/repos/CocoaPods/CocoaPods/issues/3548/comments","events_url":"https://api.github.com/repos/CocoaPods/CocoaPods/issues/3548/events","html_url":"https://github.com/CocoaPods/CocoaPods/pull/3548","id":74909179,"number":3548,"title":"[Resolver] Move side-effect of storing head pods from #specs_by_target","user":{"login":"mrackwitz","id":1389011,"avatar_url":"https://avatars.githubusercontent.com/u/1389011?v=3","gravatar_id":"","url":"https://api.github.com/users/mrackwitz","html_url":"https://github.com/mrackwitz","followers_url":"https://api.github.com/users/mrackwitz/followers","following_url":"https://api.github.com/users/mrackwitz/following{/other_user}","gists_url":"https://api.github.com/users/mrackwitz/gists{/gist_id}","starred_url":"https://api.github.com/users/mrackwitz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrackwitz/subscriptions","organizations_url":"https://api.github.com/users/mrackwitz/orgs","repos_url":"https://api.github.com/users/mrackwitz/repos","events_url":"https://api.github.com/users/mrackwitz/events{/privacy}","received_events_url":"https://api.github.com/users/mrackwitz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-05-10T12:25:55Z","updated_at":"2015-05-10T20:04:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/CocoaPods/CocoaPods/pulls/3548","html_url":"https://github.com/CocoaPods/CocoaPods/pull/3548","diff_url":"https://github.com/CocoaPods/CocoaPods/pull/3548.diff","patch_url":"https://github.com/CocoaPods/CocoaPods/pull/3548.patch"},"body":"… to #resolve.\r\nMethods, which work like getters, shouldn't have side-effects."},"comment":{"url":"https://api.github.com/repos/CocoaPods/CocoaPods/issues/comments/100690056","html_url":"https://github.com/CocoaPods/CocoaPods/pull/3548#issuecomment-100690056","issue_url":"https://api.github.com/repos/CocoaPods/CocoaPods/issues/3548","id":100690056,"user":{"login":"segiddins","id":1946610,"avatar_url":"https://avatars.githubusercontent.com/u/1946610?v=3","gravatar_id":"","url":"https://api.github.com/users/segiddins","html_url":"https://github.com/segiddins","followers_url":"https://api.github.com/users/segiddins/followers","following_url":"https://api.github.com/users/segiddins/following{/other_user}","gists_url":"https://api.github.com/users/segiddins/gists{/gist_id}","starred_url":"https://api.github.com/users/segiddins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/segiddins/subscriptions","organizations_url":"https://api.github.com/users/segiddins/orgs","repos_url":"https://api.github.com/users/segiddins/repos","events_url":"https://api.github.com/users/segiddins/events{/privacy}","received_events_url":"https://api.github.com/users/segiddins/received_events","type":"User","site_admin":false},"created_at":"2015-05-10T20:04:40Z","updated_at":"2015-05-10T20:04:40Z","body":"👍🏻🚢\n\nIn the future, feel free to just push refactors like this, as long as the specs pass\n\n-Samuel E. Giddins\n\nOn May 10, 2015, at 12:50 PM, Marius Rackwitz <notifications@github.com> wrote:\n\nI stumbled on that when working on something else and saw at as an origin of potential unwanted consequences and that I just spent a moment on double-checking that's not called twice was already too much.\n… and less code is better code.  \n\n—\nReply to this email directly or view it on GitHub.\n\n"}}