Code review comment for lp:~oubiwann/txaws/413738-method-name-cleanup

Revision history for this message
Robert Collins (lifeless) wrote :

On Thu, 2009-08-20 at 02:06 +0000, Duncan McGreggor wrote:
> On Wed, Aug 19, 2009 at 10:46 PM, Robert
> Collins<email address hidden> wrote:
> > Uhm, I'm not sure :)
> >
> > The only remaining thing was import sorting. I don't think you replied
> > to my reply on that.
>
> Hrm, well I don't know what you're sorting one... can you explain it?

the module name, and the item.

from foo import bar
from gam import alpha, beta
from gam.quux import zulu

is well sorted.

Why does sorting matter? It reduces merge conflicts - a lot. Because
when people add the same import they'll do so at the same line, and
different imports will be spread out rather than all at the bottom.

-Rob

« Back to merge proposal