Skip to content

Comments

feat(eslint): disable rules conflicting with prettier#132

Open
ekabolotina wants to merge 3 commits intoalfa-laboratory:masterfrom
ekabolotina:feature/disable-unnecessary-eslint-rules
Open

feat(eslint): disable rules conflicting with prettier#132
ekabolotina wants to merge 3 commits intoalfa-laboratory:masterfrom
ekabolotina:feature/disable-unnecessary-eslint-rules

Conversation

@ekabolotina
Copy link

Отключить те правила Eslint, которые могут быть автоматически исправлены с помощью Prettier.

Мотивация и контекст

Во время разработки нет смысла «подсвечивать» места, являющиеся проблемными с точки зрения форматирования, так как это только отвлекает от процесса разработки. Учитывая, что все эти проблемы будут автоматически исправлены с помощью Prettier, то предлагаю отключить данные проверки при помощи eslint-config-prettier.
См. также замечание в документации Prettier.

Disable rules which can be auto fixed by prettier
@ekabolotina ekabolotina requested a review from a team as a code owner June 23, 2021 10:34
Copy link
Contributor

@voronin-ivan voronin-ivan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Проверил на стабе ЕФ, в итоге линтером вообще не ловятся штуки, которые ты здесь отключаешь. Это критично, т.к. форматирование через преттир вполне себе можно скипнуть при коммите, в итоге в дев/мастер может попасть код, который совсем не соответствует стандартам.

@ekabolotina
Copy link
Author

@voronin-ivan чтобы чтобы проверить, что код соответствует стандартам в плане форматирования, запускаем prettier --check

@voronin-ivan
Copy link
Contributor

запускаем prettier --check

Почему не prettier-eslint, который используется для форматирования?
В любом случае, это нужно отразить в README.md и скриптах package.json (в этом репозитории есть псевдо-тесты).

И есть ли живой пример банковского проекта (ветки достаточно), где можно посмотреть интеграцию с этими изменениями?

@ekabolotina
Copy link
Author

запускаем prettier --check

Почему не prettier-eslint, который используется для форматирования?
В любом случае, это нужно отразить в README.md и скриптах package.json (в этом репозитории есть псевдо-тесты).

И есть ли живой пример банковского проекта (ветки достаточно), где можно посмотреть интеграцию с этими изменениями?

Добавил команду для тестирования test:format в package.json. Обновил README.md. Что касается живого проекта, — пока что такового нет.

@ekabolotina ekabolotina requested a review from voronin-ivan June 24, 2021 12:13
@Heymdall
Copy link
Member

Не соглашусь с тем, что смысла в них нет. Все правила, которые касаются форматирования у нас сейчас стоят в warn.
Это делалось осознано, так как был явный запрос от разработчиков о том, что хочется видеть то, как надо писать.

Не блокирую этот пр, но по сути я против подобного.

@ekabolotina
Copy link
Author

Не соглашусь с тем, что смысла в них нет. Все правила, которые касаются форматирования у нас сейчас стоят в warn.
Это делалось осознано, так как был явный запрос от разработчиков о том, что хочется видеть то, как надо писать.

Не блокирую этот пр, но по сути я против подобного.

ИМХО, эти правила форматирования не должны являться для разработчика ответом на вопрос «как надо писать». Они нужны только для того, чтобы кодовая база выглядела однообразно, и код было удобно читать. Эти правила не должны преследовать цель переучить людей писать правильно скобочки и отступы, т.к. это уже дело каждого разработчика (вкусовщина). А задача автоматических инструментов избавить разработчиков от этой головной боли и крови из глаз.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants