On 08/06/2015 04:40 PM, Harshal Patil wrote: I do not have anything against pull requests. Feel free to send the pull request link for the changes you want review in the cover page of the plain-text patches. They're doing nothing wrong. I was a maintainer of a Eclipse Linuxtools plug-in and they use pull requests too. It works. The reason why Kimchi (and Ginger, for that matter) uses plain-text patches in the mailing list is simplicity. You can easily quote and comment specific parts of the patch using standard email tools. I have kimchi and ginger source code in Power environments for testing and more than once I've sent patches directly from a Power host and reviewed them in pine. Using pull requests for code review implies that I need to open github, review the patch there, comment the patch there and so on. It adds one step and drags away the discussing from the ML. And, unless you want to keep refreshing the pull request URL in the browser, you'll want an email notifying that your changes went upstream. Even if github notifies that automatically to you, other people might be interested in the status of those changes too. TL;DR it is a matter of culture and Kimchi/Ginger was developed under the plain-text git-sendmail paradigm.