Opened 6 months ago

Closed 6 months ago

#21267 closed Cleanup/optimization (fixed)

pep8 cleanup: the backslash is redundant between brackets

Reported by: timo Owned by: alasdair
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 (last modified by timo)

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

Attaching an example patch to get you started.

Apply it, if you'd like:

patch -p1 -i E502.diff

Attachments (1)

E502.diff (1.7 KB) - added by timo 6 months ago.

Download all attachments as: .zip

Change History (5)

Changed 6 months ago by timo

comment:1 Changed 6 months ago by timo

  • Description modified (diff)

comment:2 Changed 6 months ago by timo

  • Description modified (diff)

comment:3 Changed 6 months ago by alasdair

  • Owner changed from nobody to alasdair
  • Status changed from new to assigned

comment:4 Changed 6 months ago by Alasdair Nicol <alasdair@…>

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

In 65d1d65d52a87bba22845bcb7c3c921a8789ec19:

Fixed #21267 -- Fixed E502 pep8 warnings

Add Comment

Modify Ticket

Change Properties
<Author field>
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.