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

Revision history for this message
Duncan McGreggor (oubiwann) 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

Awesome. Thanks for the breakdown on that.

I've updated the imports accordingly and pushed.

« Back to merge proposal