Как стать автором
Обновить

Комментарии 24

Налог — это технический долг, только наоборот?
Примерно так.
По моему скромному мнению, настройка линтера в этом случае — это попытка руководства замести под ковер некоторые проблемы в коллективе. Сложно представить, чтобы зануда со скриншота действительно испытывал сложности с чтением свойств с другим порядком аргументов, ради которых стоило бы разворачивать PR.
Да это адский пример и такого почти никогда не было. Это скорее к тому что делайте линтинг и не будет у вас такого треша.
Проблема «докопаться до запятых» не обязательно имеет отношение к проблемам в коллективе. Это очень распространённые мозговые тараканы. Bike shed problem — это, собственно, то же самое. Чем менее важен и чем более тривиален какой-то случай — тем больше людей считает возможным высказать своё очень ценное мнение по этому случаю (это не значит, что они будут его высказывать — но само наличие возможности уже означает, что таких случаев будет больше, чем случаев критики реально сложных вещей).

Все эти случаи легко решаются диктатурой сверху: сказали, что будем делать так, все теперь могут замолкнуть и делать. Линтер — это просто технический инструмент обеспечения диктатуры.
Данный случай это скорее гротеск. По поводу критики неважных вещей согласен. Где-то была старая шутка что-то в стиле «пр +3000 — 2 комента, пр +12 — 11 коментов». Диктатура часто хорошо, чтобы люди не холиварили по поводу того нужно ли ставить `;` или ASI все сам разрулит. Все пишут так как говорит линтер — все не ссорятся. Если какие-то правила не нравятся по объективным причинам можно их и глобально поменять.
Это очень распространённые мозговые тараканы.

Дихлофосить их надо, а не подкармливать)

С линтером будут холиварить на тему подходов, паттернов, инструментов, кто какую тасочку делает и куда в коде лезет. Дело не в запятых как таковых. Особенности характера)
Ну так холиварить на тему подходов и паттернов куда как лучше. Это несет какую-то смысловкую нагрузку.
Спасибо за статью, но, если честно, как-то уж слишком кратко.
Код-ревью, линтеры, автотесты и возврат технического долга — вещи само собой разумеющиеся в 2019 году.
Хотелось бы больше подробностей.
Наверняка же есть что-то ещё. Особенно в области тестирования.
Как, например, боритесь c «flaky» тестами при тестировании скриншотами?
Мы не тестируем скриншотами. Здорово если для вас это само собой разумеющиееся. Не для всех к сожалению это так. В веб студиях до сих пор пилят код, который должен быть готов вчера.
Поддержу. Все как-то банально, у большинства компаний подобный минимум соблюден, бывает даже опенсорсеры одиночки ведут свои проекты не хуже.
1k файлов, сколько у вас в штате фронтовиков, почему так много файлов?:)
У нас очень большой проект. Фронтов больше 20. Большая часть функционала доступна только для работодателей. + сейчас внедряем реакт, там куча файлов.
Мне кажется это был зарказм тк 1к файлов это немного как по мне, тем более при 20 фронтендеров.
Нет не сарказм, просто не доводилось работать с такими проектами где по фронту может быть столько логики
Современные фреймворки пропагандируют компонентный подход к разработке (что хорошо), и это влияет на количество файлов в проекте. Классический компонент это 3 файла (js/ts + html + css/sass) и сами компоненеты могут часто быть совсем небольшими и без особой логики (dump которые, просто отрисовка допустим из центрального redux-like стора).

PS 1к файлов не проблема сгенерировать двум толковым фронтендерам за год.
Где-то вы должны дойти до prettier :)
Чтоб таких ситуаций было ещё меньше.
Так дошли же.

Точно, что-то упустил этот момент.

Пишите лучше hh. А то НН можно за Нижний Новгород принять.
а типизацию используете?
Увы, пока нет. Думаем над этим вопросом.
Почему с husky перешли на lint-staged?
Они не заменяют друг друга же. Надо чтобы что-то запустило линт стейдж. Мы решили что нам не нужна лишняя зависимость. Сами на пост инсталле пишем нужный хуки в `.git/hooks/`
Зарегистрируйтесь на Хабре, чтобы оставить комментарий