Skip to content
/ django Public

Fixed #23395 -- Limited line lengths to 119 characters.#5224

Closed
dodobas wants to merge 75 commits intodjango:masterfrom
dodobas:ticket_23395
Closed

Fixed #23395 -- Limited line lengths to 119 characters.#5224
dodobas wants to merge 75 commits intodjango:masterfrom
dodobas:ticket_23395

Conversation

@dodobas
Copy link
Contributor

@dodobas dodobas commented Sep 3, 2015

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd suggest one item per line.

@timgraham
Copy link
Member

You may squash commits when updating, thanks!

@timgraham timgraham changed the title Fix lines longer than 119 characters to conform to the "coding style" Fixed #23395 -- Limited line lengths to 119 characters. Sep 3, 2015
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually 'TEST_RUNNER'/'MIDDLEWARE_CLASSES' can be removed as the compatibility checks mentioned are removed. I've done this on master so please rebase your branch.

@timgraham
Copy link
Member

Working on rebasing and addressing review comments in #5275. Thanks for getting this started.

@timgraham timgraham closed this Sep 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants