<div dir="ltr"><div dir="ltr"><div></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 29, 2019 at 5:34 PM Tom Stellard via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
As part of the migration of LLVM's source code to github, we need to update<br>
our developer policy with instructions about how to interact with the new git<br>
repository.  There are a lot of different topics we will need to discuss, but<br>
I would like to start by initiating a discussion about our merge commit<br>
policy.  Should we:<br>
<br>
1. Disallow merge commits and enforce a linear history by requiring a<br>
   rebase before push.<br></blockquote><div>I agree that this should be considered the status quo. The other options need compelling benefits to justify a change.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
2. Allow merge commits.<br></blockquote><div>
For the purposes of the canonical version of the project, I believe that
 history of side-branches (the main value of merge commits for me) is 
not of great interest.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
3. Require merge commits and disallow rebase before push.<br></blockquote><div>I do not believe #3 is enforceable (the disallow rebase part) unless if you meant to just remove the option from the GitHub UI interface.</div><div><br></div></div></div>