First of all - forget the max-line-length comment from earlier... I went
through the pep8's configuration options, and there is basically next to none,
so the overall decision will mostly need to be "keep it or drop it".
> ==== E251 =====
> Josef is used to add spaces between keyword name and its value in method
> definitions or method calls. Personally, I find it more readable according
> to PEP8 (with no space), but Josef claims the opposite :)
I can, of course, try and change my mindset, but it is true, that i find it
more readable the way I'm used to write it
> ==== E303 ====
>
> This forbids you from adding two blank lines between class methods. Another
> braindead checks.
Sadly, this can not be configured in any way.
> ==== E124 ====
>
> In a longer line of text, having the closing bracket really matching the
> opening one (the first example) is much easier on the eyes.
Agreed!
> ==== E122 =====
>
> I don't really understand the purpose of this error. If I move 'type(' to the
> third line, it goes away.
I must say, that I find the "proper" way enforced by pep8 to be a bit more
readable/understandable. It makes sense, but we can IMHO easily dismiss this
error.
> ==== E111, E113 ====
>
> I'm not sure if we can suppress certain warnings only in
> a selected block of code, but if we check for PEP8 automatically, we should
> find out. There will be more use cases like this one.
There is an option to disable checks using #noqa comment, but (sadly) not for
these errors.
All in all I'd be for configuration that ignores at least E128 and E303. My
personal preference would be also ignoring E303 and E124.
I have no strong opinion on E122, although I'd vote for keeping it, should it
ever come to it.
E111 and E113 should IMHO be kept, even if it means this kind of false
positives.
J.
_______________________________________________
qa-devel mailing list
qa-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/qa-devel
