Automatic Linting and Reformatting

INFO

Детальна інформація, що, чому і як – в коментарях задачі WTEL-6536.

WARNING

Виклик реформатування треба робити самостійно. pre-commit hook'а – немає.

TLDR;

• реформатувати все – у використовуємо npm run lint:fix
• реформатувати тільки діффи – npm run lint:fix-staged
• у Webstorm – вимкнути в Code -> Reformat File: “Optimise Imports”, “Rearrange Code”, ввімкнути eslint on save, prettier on save

Навіщо і як має працювати?

Навіщо?

Щоб не витрачати час і не морочитись з ручним реформатуванням коду. Або стилістичними коментарями в PR.

Як має працювати?

Автоматично – в тому і задумка. Якщо щось автоматично не працює і треба робити руками – кажіть.

FAQ

Чому саме так, а не інакше?

Читайте в коментарях до задачі, лінк вище.

Якщо після виконання команд створюються діффи, яких не мало би бути?

Пишіть, будем розбиратися. Ця штука потрібна саме для того, щоб таких проблем не виникало.

Якщо лінтер щось не реформатує автоматично, про що просить ревʼювер?

Пишіть. Будемо або вносити ці правила в доки, або відмовлятись від них, щоб не мати мороки з цим.

Чому не можна використовувати pre-commit hook?

Тому що з тим проблеми. Детально описав в задачі.

А CSS також має форматуватись?

Так. Prettier має виставляти по правильному порядку css properties.