feat(eslint): disable rules conflicting with prettier#132
feat(eslint): disable rules conflicting with prettier#132ekabolotina wants to merge 3 commits intoalfa-laboratory:masterfrom
Conversation
Disable rules which can be auto fixed by prettier
voronin-ivan
left a comment
voronin-ivan
left a comment
There was a problem hiding this comment.
Проверил на стабе ЕФ, в итоге линтером вообще не ловятся штуки, которые ты здесь отключаешь. Это критично, т.к. форматирование через преттир вполне себе можно скипнуть при коммите, в итоге в дев/мастер может попасть код, который совсем не соответствует стандартам.
|
@voronin-ivan чтобы чтобы проверить, что код соответствует стандартам в плане форматирования, запускаем |
Почему не prettier-eslint, который используется для форматирования? И есть ли живой пример банковского проекта (ветки достаточно), где можно посмотреть интеграцию с этими изменениями? |
Добавил команду для тестирования |
|
Не соглашусь с тем, что смысла в них нет. Все правила, которые касаются форматирования у нас сейчас стоят в warn. Не блокирую этот пр, но по сути я против подобного. |
ИМХО, эти правила форматирования не должны являться для разработчика ответом на вопрос «как надо писать». Они нужны только для того, чтобы кодовая база выглядела однообразно, и код было удобно читать. Эти правила не должны преследовать цель переучить людей писать правильно скобочки и отступы, т.к. это уже дело каждого разработчика (вкусовщина). А задача автоматических инструментов избавить разработчиков от этой головной боли и крови из глаз. |
3 participants
Отключить те правила Eslint, которые могут быть автоматически исправлены с помощью Prettier.
Мотивация и контекст
Во время разработки нет смысла «подсвечивать» места, являющиеся проблемными с точки зрения форматирования, так как это только отвлекает от процесса разработки. Учитывая, что все эти проблемы будут автоматически исправлены с помощью Prettier, то предлагаю отключить данные проверки при помощи eslint-config-prettier.
См. также замечание в документации Prettier.