On 31 July 2013 12:03, Dr. Trigon <dr.trigon@surfeu.ch> wrote:

As I mentioned - IMHO I would start by ignoring everything that our
code does not pass at the moment and then starting to improve the code
and successively remove ignores whenever possible.

My experience with fixing several files was that it's actually better to walk through the existing code, fixing pep8's complaints or disabling checks, depending on whether the check improves the code or not.

http://www.mediawiki.org/wiki/Gerrit/Code_review#Goals

"Tolerate idiosyncrasies where possible. Developers should feel that
their creative input is valued and preserved."

This is very true. But remember having a consistent style also has a big advantage: it makes reading the code easier because of the consistency.

Thanks for your input on the issue.

Merlijn