<div dir="ltr">Hello,<div><br></div><div>GitHub apparently allows fast-forward merges to trunk.  Here's an example of one:</div><div><br></div><div>  <a href="https://github.com/twisted/twisted/pull/730">https://github.com/twisted/twisted/pull/730</a></div><div><br></div><div>This doesn't seem like a good thing.</div><div><ul><li>The ticket is still open</li><li>There is no merge commit</li><li>There is no merge commit message</li><li>There are non-merge commits directly on trunk history (first parent)</li></ul></div><div>  Anyone else have an opinion?</div><div><br></div><div>Also, on the same PR, it seems like folks have trouble managing the two different ways to represent the review states: the "review" keyword in trac and the accepted/etc status on the GitHub PR.  Maybe there shouldn't be two different ways to represent this?</div><div><br></div><div>Jean-Paul</div><div><br></div></div>