{"comment":{"created_at":"2011-11-26T15:41:20Z","commit_id":"b1cc4813b1386ac6dcc969db055f9efa0f5e241c","position":null,"updated_at":"2011-11-26T15:41:20Z","user":{"gravatar_id":"6e0df11a3326dfcf8e81f05ffd3f5502","id":85518,"url":"https://api.github.com/users/schof","login":"schof","avatar_url":"https://secure.gravatar.com/avatar/6e0df11a3326dfcf8e81f05ffd3f5502?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"body":"@romul this is coming from Github \"merge button.\"  I'm not sure why\nthey don't rebase but we're experimenting with it b/c its so easy and\nwe have so many pull requests.  I have mixed feelings about whether we\nshould continue.\n\nOn Sat, Nov 26, 2011 at 7:55 AM, Roman Smirnov\n<reply@reply.github.com>\nwrote:\n> @joneslee85: We doing rebase branches before each merge. \"Merge\" commits, like this, are evil. Never again don't do so in the master branch\n>\n> ---\n> Reply to this email directly or view it on GitHub:\n> https://github.com/spree/spree/commit/b1cc4813b1386ac6dcc969db055f9efa0f5e241c#commitcomment-747627\n>","id":747768,"path":null,"line":null,"html_url":"https://github.com/spree/spree/commit/b1cc4813b1#commitcomment-747768","url":"https://api.github.com/repos/spree/spree/comments/747768"}}