Opened 12 years ago

Closed 12 years ago

#21285 closed Cleanup/optimization (fixed)

pep8 cleanup: continuation line indentation

Reported by: Tim Graham Owned by: alasdair
Component: Core (Other) Version: dev
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

Includes two errors:

  • E21 continuation line indentation is not a multiple of four
  • E122 continuation line missing indentation or outdented

To find errors, ensure you have an up-to-date master with the flake8 config in setup.cfg, then remove E121,E122 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 115.

Attaching an example patch to get you started.

Apply it, if you'd like:

patch -p1 -i E121.diff

Change History (3)

by Tim Graham, 12 years ago

Attachment: E121.diff​ added

comment:1 by alasdair, 12 years ago

Owner: changed from nobody to alasdair
Status: new → assigned

Pull request ​https://github.com/django/django/pull/1766

comment:2 by Tim Graham <timograham@…>, 12 years ago

Resolution: → fixed
Status: assigned → closed

In dfb4cb9970f86487f0aaa88c5dfcfafa31e4f430:

Fixed #21285 -- Fixed E121,E122 pep8 warnings

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