Выбор ответа Senior Линтеры и форматирование обновлено
Внедрить строгий линтер в старый проект
Вопрос
В проект на 300 тысяч строк хотят включить строгие правила, и сразу находится 4000 ошибок. Какой подход разумнее?
Попробуйте сами
Выберите вариант, и откроется разбор.
Показать ответ и разбор
Коротко
Правильный ответ: Постепенно: --fix сразу, остальное как warn или для новых файлов. Постепенное внедрение даёт пользу сразу и не блокирует команду.
Варианты ответа
- Добавить
/* eslint-disable */во все файлы и снимать постепенно - Постепенно:
--fixсразу, остальное как warn или для новых файловправильный ответ - Остановить разработку и исправить все 4000 ошибок одним большим PR
- Отказаться от линтера, раз ошибок так много в старом коде
Разбор
Постепенное внедрение даёт пользу сразу и не блокирует команду. Автоисправимое чинится одним коммитом, новые нарушения не появляются, старые уменьшаются по мере работы с кодом.
- Массовый
eslint-disableвыключает линтер, только сложнее. - Огромный PR трудно проверить, он конфликтует со всеми ветками и останавливает работу.
- Отказ от линтера оставит проблемы и дальше расти.
Ответьте сами, а не прочитайте
В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.