{"issue":{"created_at":"2011-12-10T04:50:54Z","state":"closed","updated_at":"2011-12-10T18:07:55Z","comments":4,"user":{"gravatar_id":"411e6c0c66fc71fd9ff40d267eadaf9b","id":1253094,"url":"https://api.github.com/users/MGA555","avatar_url":"https://secure.gravatar.com/avatar/411e6c0c66fc71fd9ff40d267eadaf9b?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"MGA555"},"body":"Do you think this is a good change.  It will ....","assignee":null,"id":2509846,"pull_request":{"patch_url":"https://github.com/physicsmazz/git_test/pull/1.patch","html_url":"https://github.com/physicsmazz/git_test/pull/1","diff_url":"https://github.com/physicsmazz/git_test/pull/1.diff"},"labels":[],"number":1,"title":"Remove page load tracking from GA code","html_url":"https://github.com/physicsmazz/git_test/issues/1","closed_at":"2011-12-10T11:08:53Z","url":"https://api.github.com/repos/physicsmazz/git_test/issues/1","milestone":null},"action":"created","comment":{"created_at":"2011-12-10T18:07:55Z","updated_at":"2011-12-10T18:07:55Z","user":{"gravatar_id":"76b9e62b67f91074af9fc92481ee45e8","id":1014287,"url":"https://api.github.com/users/physicsmazz","avatar_url":"https://secure.gravatar.com/avatar/76b9e62b67f91074af9fc92481ee45e8?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"physicsmazz"},"body":"I say merge because I don't know if it overwrote what I had before, or merged. \r\nI made a change to my local copy, so next time I pull in some changes, I'll see what happens.\r\n\r\n\r\n-----Original Message-----\r\nFrom: MGA555 [mailto:reply@reply.github.com] \r\nSent: Saturday, December 10, 2011 9:34 AM\r\nTo: Mike Mazzantini\r\nSubject: Re: [git_test] Remove page load tracking from GA code (#1)\r\n\r\nYou've got me confused when you say \"I assume it merge(ed)\", as you can easily tell if it did or didn't?\r\n\r\nIf I suggest a change, you of course might want to test it.  I would think merging directly into the operational copy would be desirable so that if testing is good, you have nothing more to do.  If you test on a different branch/fork, after testing you would then need to pull/push into the operational copy.  For you, I do not see that as useful.  It's more work, and an opportunity for you to forget the step. For a large organization I can see that having a use.\r\n\r\n-----Original Message-----\r\nFrom: Mike Mazzantini [mailto:reply@reply.github.com] \r\nSent: Saturday, December 10, 2011 9:20 AM\r\nTo: MGA555\r\nSubject: Re: [git_test] Remove page load tracking from GA code (#1)\r\n\r\nI also noticed that because I used a pull, it automatically merged it with my other file.\r\nI assume it merge, since I hadn't changed anything I don't know. \r\nIt probably would have made more sense to pull to a different branch or fetch. I think pull is equivalent of a fetch and merge.\r\n\r\n--- \r\nReply to this email directly or view it on GitHub:\r\nhttps://github.com/physicsmazz/git_test/pull/1#issuecomment-3091022\r\n\r\n--- \r\nReply to this email directly or view it on GitHub:\r\nhttps://github.com/physicsmazz/git_test/pull/1#issuecomment-3091077","id":3092090,"url":"https://api.github.com/repos/physicsmazz/git_test/issues/comments/3092090"}}