git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Conforming to pep8

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
May 9, 2014, 07:35 UTC
Message-ID
<536c8546ed6fd_182dd0d3100@nysa.notmuch>
In-Reply-To
<20140509072820.GD9051@wst420>
William Giokas wrote:
Show 16 quoted lines
> On Fri, May 09, 2014 at 02:18:54AM -0500, Felipe Contreras wrote:
> > William Giokas wrote:
> > > On Thu, May 08, 2014 at 11:36:29PM -0500, Felipe Contreras wrote:
> > > > William Giokas wrote:
> > > > > E401: Multi-line imports seems like something that would just be
> > > > > changing one line
> > > > 
> > > > Yes, and make the code very annoying.
> > > 
> > > It's 1 extra line in git-remote-hg, and 4 lines in git-remote-bzr.
> > 
> > Ah, it refers to the '^import *' not '^from x import'. That's fine then.
> 
> Yeah. In fact, for the mercurial stuff the `from mercurial import changegroup`
> line should be on the same line as the other `from mercurial import ...`
> line.

I think the line is too big, it should be rearranged. But every time I add something to that list I say: I'll clean it up later =/

-- 
Felipe Contreras
Previous: William GiokasNext: William Giokas
Message 10 of 14 in “Conforming to pep8”
  1. William GiokasMay 9, 2014
  2. Jonathan NiederMay 9, 2014
  3. Michael HaggertyMay 9, 2014
  4. Felipe ContrerasMay 9, 2014
  5. William GiokasMay 9, 2014
  6. Felipe ContrerasMay 9, 2014
  7. William GiokasMay 9, 2014
  8. Felipe ContrerasMay 9, 2014
  9. William GiokasMay 9, 2014
  10. Felipe ContrerasMay 9, 2014
  11. William GiokasMay 9, 2014
  12. W. Trevor KingMay 9, 2014
  13. Felipe ContrerasMay 9, 2014
  14. John KeepingMay 9, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.