<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Mon, Oct 28, 2013 at 3:18 AM, Mark McLoughlin <span dir="ltr"><<a href="mailto:markmc@redhat.com" target="_blank">markmc@redhat.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"><div class=""><div class="h5">On Sun, 2013-10-27 at 21:50 -0400, Monty Taylor wrote:<br>

> Hey all!<br>
><br>
> We're adding a little bit of code to pbr to make the auto-generated<br>
> ChangeLog files a bit more useful. Currently, they are just the git<br>
> changelog, which is kinda useless. So we wrote this:<br>
><br>
> <a href="https://review.openstack.org/#/c/52367/" target="_blank">https://review.openstack.org/#/c/52367/</a><br>
><br>
> which produces output like this:<br>
><br>
> <a href="http://paste.openstack.org/show/49999" target="_blank">http://paste.openstack.org/show/49999</a>  # on a tag<br>
> and<br>
> <a href="http://paste.openstack.org/show/50001" target="_blank">http://paste.openstack.org/show/50001</a>  # not on a tag<br>
><br>
> It underscores the need for commit subjects to communicate something,<br>
> which is a good thing.<br>
><br>
> With that there, it makes changes like:<br>
> * Updated from global requirements<br>
> and<br>
> * Update my mailmap<br>
><br>
> Kinda lame in the changelog. So we were thinking - what if we recognized<br>
> one or more subject tags to skip things going into the ChangeLog file.<br>
> My first thought was:<br>
><br>
> NIT:  # use this for tiny commits that are not really worth changelogging<br>
><br>
> and<br>
><br>
> AUTO: # use for commits generated by our machines, such as the global<br>
> requirements sync or the Translations sync.<br>
><br>
> What do people think? Should we bother? Are those good? Would something<br>
> else be better? It's sort of an opt-in feature, so adding it SHOULDN'T<br>
> bork too many people.<br>
<br>
</div></div>So long as it isn't so SHOUTY it could work out nicely :)<br></blockquote><div><br></div><div>+1, I like the approach to nit / auto so long as it isn't strict. For example,</div><div><br></div><div>  ''.join(c for c in summary.split()[0] if c.isalnum()).upper() in (NIT, AUTO)</div>
<div><br></div><div>would support "NIT:", "(nit) ...", "Nit- ...", etc.</div><div><br></div><div>I'd also be happy with a footer, since a lot of nit / auto commits don't have bugs / bp's to reference... but I don't know what that would look like as a footer?</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">
<br>
Getting these ChangeLogs published is goodness - the more the more<br>
people see their one-line message around the place, the more people<br>
they'll make an effort to write a decent one.<br>
<br>
Cheers,<br>
Mark.<br>
<div class=""><div class="h5"><br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div><br></div>-Dolph
</div></div>