{"action":"created","issue":{"url":"https://api.github.com/repos/iojs/io.js/issues/574","labels_url":"https://api.github.com/repos/iojs/io.js/issues/574/labels{/name}","comments_url":"https://api.github.com/repos/iojs/io.js/issues/574/comments","events_url":"https://api.github.com/repos/iojs/io.js/issues/574/events","html_url":"https://github.com/iojs/io.js/issues/574","id":55300596,"number":574,"title":"Fix the broken collaborator merge process","user":{"login":"geek","id":441346,"avatar_url":"https://avatars.githubusercontent.com/u/441346?v=3","gravatar_id":"","url":"https://api.github.com/users/geek","html_url":"https://github.com/geek","followers_url":"https://api.github.com/users/geek/followers","following_url":"https://api.github.com/users/geek/following{/other_user}","gists_url":"https://api.github.com/users/geek/gists{/gist_id}","starred_url":"https://api.github.com/users/geek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geek/subscriptions","organizations_url":"https://api.github.com/users/geek/orgs","repos_url":"https://api.github.com/users/geek/repos","events_url":"https://api.github.com/users/geek/events{/privacy}","received_events_url":"https://api.github.com/users/geek/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-01-23T16:20:41Z","updated_at":"2015-01-23T19:05:32Z","closed_at":null,"body":"Node has a large amount of simple changes that take forever to get merged, if at all.  The same thing is starting to occur in iojs land, with simple PRs piling up.  A large reason for this is the upfront time suck that is the collaborator merge process (https://github.com/iojs/io.js/blob/v1.x/COLLABORATOR_GUIDE.md#technical-howto). \r\n\r\nSolution: Allow core contributors to click the green button for doc/readme changes that have a clean commit message.  Use the PR itself to track the review.  \r\n\r\nI would really like to see core contributors be able to merge *any* pull request that has been rebased and has a clean commit message.\r\n\r\nThe upfront time cost of the current merging process far outweighs any time savings you will see when you are tracking down an issue.  Mainly because the former occurs 100% of the time and the latter rarely occurs. "},"comment":{"url":"https://api.github.com/repos/iojs/io.js/issues/comments/71246383","html_url":"https://github.com/iojs/io.js/issues/574#issuecomment-71246383","issue_url":"https://api.github.com/repos/iojs/io.js/issues/574","id":71246383,"user":{"login":"jonathanong","id":643505,"avatar_url":"https://avatars.githubusercontent.com/u/643505?v=3","gravatar_id":"","url":"https://api.github.com/users/jonathanong","html_url":"https://github.com/jonathanong","followers_url":"https://api.github.com/users/jonathanong/followers","following_url":"https://api.github.com/users/jonathanong/following{/other_user}","gists_url":"https://api.github.com/users/jonathanong/gists{/gist_id}","starred_url":"https://api.github.com/users/jonathanong/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jonathanong/subscriptions","organizations_url":"https://api.github.com/users/jonathanong/orgs","repos_url":"https://api.github.com/users/jonathanong/repos","events_url":"https://api.github.com/users/jonathanong/events{/privacy}","received_events_url":"https://api.github.com/users/jonathanong/received_events","type":"User","site_admin":false},"created_at":"2015-01-23T19:05:32Z","updated_at":"2015-01-23T19:05:32Z","body":"> does the blame page solve the issue of understanding who made what change and when? Thats what I use when I need to track down the author of a code change.\r\n\r\nblame page is fine with amend/commits. find LOC change -> go to commit -> read commit description + links.\r\n\r\nanother issue is that when collaborators focus on the big green button, they try to get the collaborators to structure the PR __exactly__ how they want w/ style changes, commit messages, etc. this makes it harder for new collaborators to collaborate and it's easier for the maintainers to amend/commit locally with their own minor fixes while still giving the original author credit. an example of this is how express works with @dougwilson. "}}