<div dir="ltr">Imo, it creates more problems than profits. If someone wants to change the code and run tests -> use git repositories. Prepared python package is not about this.<br><div><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">2018-02-19 17:57 GMT+02:00 Doug Hellmann <span dir="ltr"><<a href="mailto:doug@doughellmann.com" target="_blank">doug@doughellmann.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">IIRC we started doing that so that consumers building their own packages<br>
can run the tests for the packages easily. I don't know how many people<br>
are doing that, and apparently at least some downstream consumers aren't<br>
packaging everything anyway so they couldn't run those tests.<br>
<br>
Excerpts from Andrey Kurilin's message of 2018-02-19 17:39:11 +0200:<br>
<div class="HOEnZb"><div class="h5">> Can someone explain me the reason for including "tests" module into<br>
> packages?<br>
><br>
> 2018-02-19 17:00 GMT+02:00 Michael Bayer <<a href="mailto:mbayer@redhat.com">mbayer@redhat.com</a>>:<br>
><br>
> > Hi list -<br>
> ><br>
> > Apparently Cinder was misled by my deprecations within the<br>
> > oslo_db.sqlalchemy.test_base package of DbFixture and DbTestCase, and<br>
> > in <a href="https://review.openstack.org/#/c/522290/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/522290/</a> the assumption was made<br>
> > that these should be imported from oslo_db.tests.sqlalchemy.    This<br>
> > is an immense mistake on my part that I did not expect people to go<br>
> > looking for the same names elsewhere in private packages and now we<br>
> > have a serious downstream issue as these modules are not packaged, as<br>
> > well as the possibility that the oslo_db.tests. package is now locked<br>
> > in time and I have to add deprecations there also.<br>
> ><br>
> > If anyone knows of projects (or feels like helping me search) that are<br>
> > importing *anything* from oslo_db.tests these must be reverted ASAP.<br>
> ><br>
> > ______________________________<wbr>______________________________<wbr>______________<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.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
> > <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
> ><br>
><br>
<br>
______________________________<wbr>______________________________<wbr>______________<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.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Best regards,<br>Andrey Kurilin.<br></div></div>
</div></div></div></div></div></div>