Skip to content
  • Yann E. MORIN's avatar
    Revert ".flake8: fix check for 80/132 columns" · 3b10ee39
    Yann E. MORIN authored and Thomas Petazzoni's avatar Thomas Petazzoni committed
    Commit 7d17ae2a (.flake8: fix check for 80/132 columns) introduced a
    difference in how flake8 behaves between the automatic checks done in
    the CI, where the maximum line length is 132, and the local checks,
    where the maximum line length is 80.
    
    The rationale at the time was that we recommend 80 char lines, but that
    we accept 132 when it makes sense for readability.
    
    However, this is very annoying when running flake8 locally, because of
    two reasons:
    
     1. human reviews on python scripts have not been as thorough as we did
        expect; indeed, we've let a lot of long lines slip through; this
        causes a lot of spurious failures that hide away the actual errors;
    
     2. when hacking on a python script, the issues reported will not be
        caused by the current changes, so the many reported failures
        actually hide away the newly introduced issues.
    
    Additionally, our 'make check-flake8' rule already enforces the 132-char
    limit, and the issues reported are differ...
    3b10ee39