Fixed #21288 -- Fixed E126 pep8 warnings #1780

Closed
wants to merge 1 commit into from

2 participants

@alasdairnicol

No description provided.

@alasdairnicol

I wasn't sure about this one. Moving { to the next line allows us to indent the dictionary items

@alasdairnicol

I've gone for long lines in a few places, because I think it's more readable.

@alasdairnicol

It was tricky to try and keep this readable. Put the entire tuple on one line as suggested by Tim in the ticket.

@alasdairnicol

Moved the comment above dictionary items so that key and beginning of value are on same line

@alasdairnicol

There's a few changes like this where the indentaion for %-formatting args has been removed. Maybe long lines would be better in cases like this.

@timgraham
Django member

merged in b289fcf - thanks!

@timgraham timgraham closed this Oct 21, 2013
@alasdairnicol alasdairnicol deleted the alasdairnicol:ticket_21288 branch Oct 21, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment