<div dir="ltr"><div><div><div>I agree with Maciej<br><br></div>1. Simple change in CommitMessage shouldn't spin up CI check<br></div>2. There should be simple task where we check "Implements: blueprint" or "Closes-Bug:" or Related-Bug:" to set -1 automatically<br></div>3. There can be additional checks like Short Summary should be 50 or less symbols. Long summary should be wrapped to 80 symbols<br><br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr">--<br>
Best regards,<br>
Sergii Golovatiuk,<br>
Skype #golserge<br>
IRC #holser<br></div></div></div>
<br><div class="gmail_quote">On Wed, Jul 29, 2015 at 10:58 AM, Aleksandr Didenko <span dir="ltr"><<a href="mailto:adidenko@mirantis.com" target="_blank">adidenko@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><span class=""><div>Hi,<br><br>> I think that checking commit message compliance to commit message 
guidelines (for example ending the first line with dot) is part of CI 
jobs, and they will vote -1 if message is wrongly structured.<br><br></div><h3><span name="Maciej Kwiek"><br></span></h3></span>Maciej, we don't have such checks at the moment. You can craft any commit message you want and it will not cause any problems with CI. I think, reviewers could do the job on commit message verification and they already do this :)<br><br></div>Regards,<br></div>Alex<br><br></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Wed, Jul 29, 2015 at 11:23 AM, Sergey Vasilenko <span dir="ltr"><<a href="mailto:svasilenko@mirantis.com" target="_blank">svasilenko@mirantis.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">-1 to Maciej<div>+1 to Sergii<span><font color="#888888"><br></font></span></div><span><font color="#888888"><div><br><div class="gmail_extra"><br clear="all"><div><div><div dir="ltr"><br><div>/sv</div></div></div></div>
<br></div></div></font></span></div>
<br></div></div><span class="">__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></span></blockquote></div><br></div>
<br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div>