Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Extend flake8 line length limit #191

Closed
kokosing opened this issue Jun 29, 2016 · 2 comments
Closed

Extend flake8 line length limit #191

kokosing opened this issue Jun 29, 2016 · 2 comments

Comments

@kokosing
Copy link
Contributor

Having line length limit to 80 characters is very inconvenient and causes a lot of lines to broken into multiple lines.

I would extend it to 120, what do you think?
@ArturGajowy
Copy link

LGTM. 120 is the only right value for this limit ;P

@ebd2
Copy link
Contributor

ebd2 commented Jun 29, 2016

Sounds like a good plan to me.

kokosing added a commit to Teradata/presto-admin that referenced this issue Jun 30, 2016
@kokosing
Codestyle - let line length limit to be 120 chars
a7ff3e1 
Fixes prestodb#191
@kokosing kokosing mentioned this issue Jun 30, 2016
Merged
kokosing added a commit to Teradata/presto-admin that referenced this issue Jul 1, 2016
@kokosing
Codestyle - let line length limit to be 120 chars
14d6a46 
Fixes prestodb#191

Pull request: prestodb#198
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ArturGajowy @kokosing @ebd2