<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 21, 2016 at 5:29 AM, Michael Vogt <span dir="ltr"><<a href="mailto:michael.vogt@canonical.com" target="_blank">michael.vogt@canonical.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
So to summarize, I really like the proposal for more standardized<br>
merge commit messages. I would suggest to move the affected packages<br>
out of the summary into the body and encourage the use of tags like<br>
"LP: ", "Thanks: ". This would make releasing with sensible changelogs<br>
much easier for me.<br></blockquote><div><br></div><div>I would prefer to keep the package list in the summary, even if it cuts down on space a bit. The reason is that it gives much useful context for the summary sentence itself For example, reading through the commits list here makes it clear that committers naturally think about a local context when cooking the summary:</div><div><br></div><div>    <a href="https://github.com/ubuntu-core/snappy/commits" target="_blank">https://github.com/ubuntu-core/snappy/commits</a><br></div><div><br></div><div>The package list qualifies that context in a very useful way.</div><div><br></div><div>About the additional metadata, "Thanks:" sounds great as it gives another incentive to a good practice.</div><div><br></div><div>I'm not so sure about "Affects:" though. It seems to the wrong end of it, in the sense that it's impossible to have an exhaustive list of who's affected by something, and it will often be missed. I'd prefer to keep the bug as the place that tracks who's affected by a given issue, as it has more room and may be edited after the commit is made.</div><div><br></div><div>I'd also keep the term as "Closes:" and "Updates:" rather than "LP:", as that's more clearly indicating whether it's just related to the issue or a fix for it.</div><div><br></div><div>How does that sound?</div><div><br></div><div><br></div></div><div>gustavo @ <a href="http://niemeyer.net" target="_blank">http://niemeyer.net</a></div>
</div></div>