#21298 closed Cleanup/optimization (fixed)

pep8 cleanup: expected 1 blank line, found 0

Reported by: timo Owned by: nobody
Component: Core (Other) Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

To find errors, ensure you have an up-to-date master with the flake8 config in setup.cfg, then remove E301 from the ignore list there. You can then run flake8 from the directory with setup.cfg in it to list all errors - there should be about 200.

Note I also added ./django/utils/six.py to the ignore list since it had some of these errors but we don't care about fixing it since it's mostly a copy of a 3rd party library.

Attaching an example patch to get you started.

Apply it, if you'd like:

patch -p1 -i E301.diff

Attachments (1)

E301.diff (4.7 KB) - added by timo 22 months ago.

Download all attachments as: .zip

Change History (3)

Changed 22 months ago by timo

comment:2 Changed 22 months ago by Alasdair Nicol <alasdair@…>

  • Resolution set to fixed
  • Status changed from new to closed

In c3aa2948c6c14862407501290571f858ccf45b07:

Fixed #21298 -- Fixed E301 pep8 warnings

Note: See TracTickets for help on using tickets.
Back to Top