Opened 11 years ago

Closed 11 years ago

#21287 closed Cleanup/optimization (fixed)

pep8 cleanup: closing bracket does not match indentation of opening bracket's line

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

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

Attaching an example patch to get you started.

Apply it, if you'd like:

patch -p1 -i E123.diff

Attachments (1)

E123.diff (3.2 KB ) - added by Tim Graham 11 years ago.

Download all attachments as: .zip

Change History (4)

by Tim Graham, 11 years ago

Attachment: E123.diff added

comment:1 by alasdair, 11 years ago

Owner: changed from nobody to alasdair
Status: newassigned

comment:3 by Alasdair Nicol <alasdair@…>, 11 years ago

Resolution: fixed
Status: assignedclosed

In a800036981c6fea8eb3dac22467965c71af05d29:

Fixed #21287 -- Fixed E123 pep8 warnings

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