<div dir="ltr"><div>[Armchair observer...]</div><div><br></div><div>If any merges are allowed, they should be rare, have recent parent commits, or the history graph becomes useless.</div><div><br></div>4. Each reviewed bug / feature must be rebased onto the current "known good" commit, merged into a "probably good" commit, tested by build bots, and only then pushed to trunk. Keeping trunk's history more usable, with most bad patches reworked and resubmitted instead of reverted.<div><br><div>5. When a new feature is broken up into a patch series, the series should be rebased then immediately merged to help identify the individual patches in the history graph.</div></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 30 Jan 2019 at 09:04, 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">Hi,<br>
<br>
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>
<br>
2. Allow merge commits.<br>
<br>
3. Require merge commits and disallow rebase before push.<br>
<br>
I'm going to propose that if we cannot reach a consensus that we<br>
adopt policy #1, because this is essentially what we have now<br>
with SVN.<br>
<br>
What does everyone think?<br>
<br>
Thanks,<br>
Tom<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>