On 10/08/2015 11:39 AM, Lukas Slebodnik wrote:
On (08/10/15 10:33), Pavel Reichl wrote:
>
>
> On 10/08/2015 09:32 AM, Lukas Slebodnik wrote:
>>
>> Pavel R.,
>> could these patches used with your tests for local override?
>>
>> LS
>>
>
> I'll have to amend the patches, but that was agreed to. Seems that functionality
I need is there so fine by me.
Nikolai,
please fix pep8 warnings in first 5 patches so we can push them
and unblock other patches.
I'm doing that right now. However, should we really comply with the following
warnings?
E126 continuation line over-indented for hanging indent
E131 continuation line unaligned for hanging indent
E221 multiple spaces before operator
E241 multiple spaces after ','
E251 unexpected spaces around keyword / parameter equals
These complain about whitespace which is used to try to format the code better
for humans and is subjective. Do we agree with them? Or can we keep our
formatting?
Nick