<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Mar 13, 2015 at 4:05 PM, Marco Martin <span dir="ltr"><<a href="mailto:notmart@gmail.com" target="_blank">notmart@gmail.com</a>></span> wrote:<br><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"><span>On Friday 13 March 2015 10:12:24 David Faure wrote:<br>
> The goal of the CHANGELOG line is to describe the change from the point of<br>
> view of a developer using the framework, or from the point of view of a user<br>
> running this code, while the rest of commit messages are often written from<br>
> the implementation point of view, as in the example above (which is fine,<br>
> as long as there's a CHANGELOG line further below).<br>
><br>
> But if I don't care for the changelog, I'm happy to keep writing "..." :-)<br>
<br>
</span>To everybody(and myself):<br>
please follow this advice, on any mayor change especially if needs a bit of<br>
back and forth in reviews, let's try to add CHANGELOG: in the commit, mkay?<br></blockquote><div><br></div><div>Not just "major changes". Basically all commits!</div><div><br></div><div>To quote David Faure: </div><div>--------------------</div><div><span style="font-size:12.8000001907349px;background-color:rgb(255,255,255)">CHANGELOG</span><span style="font-size:12.8000001907349px"> is for both bugfixes and features. Readers of </span><span style="font-size:12.8000001907349px;background-color:rgb(255,255,255)">changelogs</span><span style="font-size:12.8000001907349px"> are interested in both.</span><br style="font-size:12.8000001907349px"><span style="font-size:12.8000001907349px">Just not for trivial stuff (fixing after a previous broken commit, code style changes, etc.).</span><br></div><div>--------------------<span style="font-size:12.8000001907349px"><br></span></div><div><br></div><div>which comes from: <a href="https://gerrit.vesnicky.cesnet.cz/r/#/c/233/">https://gerrit.vesnicky.cesnet.cz/r/#/c/233/</a></div><div><br></div><div>Perhaps we need a "changelog police", David? ;-)</div><div><br></div><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">
<span><font color="#888888"><br>
--<br>
Marco Martin<br>
</font></span><div><div>_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org" target="_blank">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
</div></div></blockquote></div><br></div></div>