Skip to content

Conversation

okainov
Copy link
Collaborator

@okainov okainov commented Nov 9, 2015

@kirill-kornyakov @AoD314, пожалуйста, позвольте нам увеличить допустимое число символов в строке с 79 до 120.

@kirill-korniakov
Copy link
Member

А зачем?

@okainov
Copy link
Collaborator Author

okainov commented Nov 10, 2015

Хороший вопрос, на самом деле. Скорее всего, простой ответ "очень неудобно" вас не устроит, поэтому более развернуто:
79\80 символов - это исторический лимит на число символов в строке, вызванный физическим ограничением консоли, в которой ширина "окна" была именно 80 символов, а код, разделенный переносом строки, читался хуже. Сейчас никто консолью не пользуется, у многих широкоформатные мониторы с большим разрешением, многие современные IDE (как минимум, Pycharm и другие продукты JetBrains) имеют лимит в 120 символов, равно как и некоторые код стайл гайды (PHP, Maven, JavaRanch), . Потому что на мониторе среднестатистического программиста 80 - это, если повезет, четверть экрана. Остальное место пустует. Естественно, это не означает, что можно писать строки кода во весь 24"-монитор длиной, но можно следовать современным реалиям.
Код стайл гайды не прибиты гвоздями и зачастую в них написано "можно отступать, но в любом случае придерживайтесь выбранного стиля". Поэтому считаю, что принципиальной разницы с академической точки зрения не будет, придерживаться ли такого или другого стиля. Хотя, без сомнений, с 80 символами давление стиля ощущается на себе очень жестоко, когда почти перед каждым коммитом оказывается хотя бы одна строка, вылезающая за этот предел.

@kirill-korniakov
Copy link
Member

Я предлагаю взять минимум из 120 и того, что влезает в окно code review на GitHub. Не помню сколько именно, что-то около 100, можете вычислить сами.

@AoD314
Copy link
Member

AoD314 commented Nov 10, 2015

точной границы не нашел, у меня убралось 115 максимум, так что предлагаю остановится на 110

@okainov
Copy link
Collaborator Author

okainov commented Nov 10, 2015

Поправил на 110, визуальный редактор в Гитхабе еще новую строчку создал, но не критично.

@okainov okainov changed the title Увеличение числа символов в строке до 120 Увеличение числа символов в строке до 110 Nov 10, 2015
@kirill-korniakov
Copy link
Member

👍

okainov added a commit that referenced this pull request Nov 10, 2015
Увеличение числа символов в строке до 110
@okainov okainov merged commit 7b38ab3 into master Nov 10, 2015
@okainov okainov deleted the increase-flake8-line-length branch November 10, 2015 19:16
@timonov
Copy link
Contributor

timonov commented Nov 11, 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.

4 participants