<div dir="ltr">My +1 on both points.<div>Even more: project tests should enforce these style rules like it's done in other projects.</div><div><br><div style>I think those should be filed as wish-list bugs at least if not a blueprints.</div>
<div style><br></div><div style><br></div><div style>Eugene.</div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jun 6, 2013 at 12:44 PM, Tatiana V. Mazur <span dir="ltr"><<a href="mailto:tmazur@mirantis.com" target="_blank">tmazur@mirantis.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello!<br>
<br>
Recently I had to upload some patch sets and rebase some branches for this purpose. It appears that rebasing on changed master branch is not a trivial task. You know, when nearly each Horizon module includes a total mash of comma separated imports, that's really not a trivial task. Problems start when you add some method from the same module in an existing import statement like here:<br>

<br>
from .views import (IndexView, CreateView, EditAttachmentsView, DetailView,<br>
                    CreateSnapshotView)<br>
<br>
or even here:<br>
<br>
from .views import IndexView<br>
from .views import AddPoolView, AddMemberView, AddMonitorView, AddVipView<br>
from .views import (UpdatePoolView, UpdateMemberView,<br>
                    UpdateVipView, UpdateMonitorView)<br>
from .views import PoolDetailsView, VipDetailsView<br>
from .views import MemberDetailsView, MonitorDetailsView<br>
from .views import AddPMAssociationView, DeletePMAssociationView<br>
<br>
All patch sets including this kind of changes can't be merged automatically. You have to rebase manually, just because of a few imports. I was wondering if there is a local Horizon standard for importing and tried to find it in HACKING.rst (like all standards are described in other projects). And it appears there's no HACKING.rst in Horizon! That's why I have a few questions:<br>

<br>
1. Does it make sense to implement all imports uniformly like that is done in other projects?<br>
<br>
Something like this would be nice:<br>
<br>
from quantum.api.extensions import (<br>
    ExtensionMiddleware,<br>
    PluginAwareExtensionManager,<br>
)<br>
from quantum.common import config<br>
from quantum.extensions import (<br>
    credential,<br>
    qos,<br>
)<br>
<br>
Or just one import per line. That would solve merging problems and code would look much better.<br>
<br>
2. Does it make sense to add HACKING.rst in Horizon?<br>
<br>
I think yes. We could describe standards there and we would never get such kind of rebase and merge problems.<br>
<br>
What do you think about it? I could do it if you find it reasonable .<br>
<br>
<br>
Kind regards,<br>
Tatiana<br>
<br>
______________________________<u></u>_________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
</blockquote></div><br></div>