Skip to content

Pep8 errors fixed #9

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.

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

Conversation

misamoylov
Copy link

No description provided.

@stalkerg
Copy link
Contributor

Restriction 80 chars per line is too old.

@stalkerg
Copy link
Contributor

Pep8:
Some teams strongly prefer a longer line length. For code maintained exclusively or primarily by a team that can reach agreement on this issue, it is okay to increase the nominal line length from 80 to 100 characters (effectively increasing the maximum length to 99 characters), provided that comments and docstrings are still wrapped at 72 characters.

@funbringer funbringer merged commit 1e41f4d into postgrespro:master Sep 16, 2016
@funbringer
Copy link
Collaborator

Давайте на будущее решим так: 80 символов на строку - это и правда мало, я не вижу смысла в таком драконовском ограничении. Лучше мы немного подрегулируем настройки flake и т.п., потому что на читабельность это влияет не лучшим образом.

@misamoylov
Copy link
Author

Ок. Давай на 99 остановимся.

@misamoylov misamoylov deleted the refactoring_for_strings_format_instead_percents branch September 19, 2016 10:35
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.

3 participants