<div dir="ltr">This touches on one of my big pet peeves, in the commit message one should: <div><br></div><div>"<b style="color:rgb(51,51,51);font-family:'Arial Unicode MS',Arial,sans-serif;font-size:14px;line-height:20px">Describe <i>why</i> a change is being made.</b></div>

<p style="margin:0px 0px 10px;color:rgb(51,51,51);font-family:'Arial Unicode MS',Arial,sans-serif;font-size:14px;line-height:20px">A common mistake is to just document how the code has been written, without describing /why/ the developer chose to do it that way. By all means describe the overall code structure, particularly for large changes, but more importantly describe the intent/motivation behind the changes."</p>

<p style="margin:0px 0px 10px;color:rgb(51,51,51);font-family:'Arial Unicode MS',Arial,sans-serif;font-size:14px;line-height:20px">Actually,  <a href="https://wiki.openstack.org/wiki/GitCommitMessages#Information_in_commit_messages">https://wiki.openstack.org/wiki/GitCommitMessages#Information_in_commit_messages</a> has a really great description of what a commit message should include and why, but rarely due.</p>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 8, 2013 at 12:03 PM, Mac Innes, Kiall <span dir="ltr"><<a href="mailto:kiall@hp.com" target="_blank">kiall@hp.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Taking a total guess here... But, I reckon that it's usefulness now that<br>
oslo.* are being spun off into "real projects" means that the shared<br>
dependency is becoming a problem.<br>
<br>
e.g. oslo.config and oslo.messaging shouldn't both include it, so should<br>
a whole oslo.exception library be created for what is essentially 1<br>
shared base exception class? I personally don't think so.<br>
<br>
Thanks,<br>
Kiall<br>
<div><div class="h5"><br>
On 08/08/13 19:12, Joshua Harlow wrote:<br>
> Hi recently I was working with some cinder code and noticed that<br>
> oslo.exception is being dropped from it (and other projects).<br>
><br>
> It seems connected back to this bug:<br>
> <a href="https://bugs.launchpad.net/oslo/+bug/1208734" target="_blank">https://bugs.launchpad.net/oslo/+bug/1208734</a><br>
><br>
> <a href="https://github.com/openstack/oslo-incubator/blob/master/MAINTAINERS#L118" target="_blank">https://github.com/openstack/oslo-incubator/blob/master/MAINTAINERS#L118</a><br>
><br>
> I'm just wondering if there is any more reason for why it is obsoleted,<br>
> is it being replaced? Was there just no one supporting it? Was it<br>
> decided that its not useful?<br>
><br>
> Thanks much,<br>
><br>
> Josh<br>
<br>
</div></div>_______________________________________________<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>
</blockquote></div><br></div>