Spreedly Blog: "Merge pull request" Considered Harmful

url: http://blog.spreedly.com/2014/06/24/merge-pull-request-considered-harmful/#.U8YKuhaczwJ

Nathanial, maintainer of ActiveMerchant's open source repository, talks about the downsides of "merge pull request" button on the web interface of Github and presents an alternative.

The most interesting part of the post is the alternative workflow that allows for handling the case where the pull request's code needs work before it can be merged.

A nice side effect is that the commit history isn't full of the redundent "merged pull request" commit lines.

Discuss This Topic

There are 0 comments in this discussion.

join this discussion

Creative Commons Attribution-Share Alike 3.0 New Zealand License
Spreedly Blog: "Merge pull request" Considered Harmful by Walter McGinnis is licensed under a Creative Commons Attribution-Share Alike 3.0 New Zealand License