On 10/ 2/13 06:21 AM, Xiaobo Shen wrote:
Here is a fixed one:
That looks good.
If you could please use two blank lines between class definitions, line
407 and 417 of api_errors.py and move "e.filename" up to line 635 on
repository.py (because it'll fit within the 80-column limit), then I'm
happy with the changes.
[these are all style nits: I wish we had PEP8 checks already, which
would make life easier for new and experienced developers 17308336 :-) ]
[pkg-discuss] Re: Review request 17376567 Unable to use pkgrecv - failing with [Errno 22] Invalid argument