Почему ошибки ТС отличаются когда запускаешь сам и когда в lint-staged?

Question

[GaserV]

Такая проблемка. В package-json есть:

"lint-staged": {
    "src/**/*.{ts,tsx}": [
      "eslint --cache --fix",
      "tsc --pretty --noEmit"
    ]
  },

И есть несколько проблем, которые хотелось бы решить:

1. Когда запускаешь команду tsc --pretty --noEmit в терминалке и когда срабатывает в lint-staged - выбрасываются абсолютно разные ошибки. В одном случае одно в другом другое
2. Как можно настроить проверку ТС только на затронутые файлы а не на весь проект?

Comments

[Александр]

Запускайте через npx оба варианта

[WbICHA]

Потому что линт стейдж игнорит тсконфиг.
https://github.com/lint-staged/lint-staged/issues/825

[WbICHA]

А вообще хз зачем этот линт стейдж юзать, никаких реальных плюсов он не даёт, а шанс запушить сломанный код даёт.

[Aetae]

WbICHA, на большом проекте "линт всего" занимает дохрена времени же.

[WbICHA]

Aetae, это насколько проект должен быть большим, чтобы линтинг таким долгим был? А если проект настолько большой, то почему это монолит тогда?

[Aetae]

WbICHA, а оно и не монолит, всё равно дохрена. Кровавый энтерпрайз. Так получилось что я никогда по всяким лендингам-одностраничникам не работал, мб там проще.
Дохрена это не часы, если что, но и минута перед каждым коммитом - напряг.

[WbICHA]

Aetae, ну хз, прям с большими проектами я, наверно, не работал, но проблем со скоростью как-то ни разу не было, хотя у меня плагинов там штук 10. Впрочем, надо будет как-нибудь до 9ой версии обновиться, он, вроде, быстрее, но не все плагины обновились...
И, пожалуй, самое главное, я использую только препуш.) Потому что даже преттиер ждать для каждого коммита для меня слишком долго, да и зачем, если все изменения всё равно лишь локально.)

[Александр]

WbICHA, Знаю, что если юзать некоторые плагины (напр. eslint-import с правилом no-cycle), то время линтовки возрастает заметно.

[WbICHA]

Александр, сейчас проверил с цикличным и без, что с ним 22+- секунды, что без него 22+- секунды. Так что хз.)

[Александр]

WbICHA, Прошло конечно очень много времени, но вот мне снова попался кейс.
Имеем:
1. циклические зависимости и включенный import/no-cycle

Rule                                 | Time (ms) | Relative
:------------------------------------|----------:|--------:
import/no-cycle                      | 11511.688 |    34.7%
@typescript-eslint/naming-convention |  8997.119 |    27.1%
prettier/prettier                    |  7475.841 |    22.5%
import/namespace                     |  1279.176 |     3.9%
react/no-unused-prop-types           |   487.986 |     1.5%
import/order                         |   459.598 |     1.4%
react/no-direct-mutation-state       |   374.867 |     1.1%
import/no-extraneous-dependencies    |   303.322 |     0.9%
react/require-render-return          |   238.389 |     0.7%
react/jsx-no-target-blank            |   114.757 |     0.3%

2. циклические зависимости и выключенный import/no-cycle:

Rule                                 | Time (ms) | Relative
:------------------------------------|----------:|--------:
@typescript-eslint/naming-convention |  9119.487 |    36.8%
prettier/prettier                    |  7412.667 |    29.9%
import/namespace                     |  4362.292 |    17.6%
react/no-unused-prop-types           |   479.613 |     1.9%
import/order                         |   466.176 |     1.9%
react/no-direct-mutation-state       |   358.955 |     1.4%
import/no-extraneous-dependencies    |   315.997 |     1.3%
react/require-render-return          |   247.203 |     1.0%
react/jsx-no-target-blank            |   117.753 |     0.5%
object-shorthand                     |    99.188 |     0.4%

3. без циклических и включенный import/no-cycle:

Rule                                 | Time (ms) | Relative
:------------------------------------|----------:|--------:
import/no-cycle                      | 11696.468 |    34.6%
@typescript-eslint/naming-convention |  9126.938 |    27.0%
prettier/prettier                    |  7624.741 |    22.6%
import/namespace                     |  1268.783 |     3.8%
import/order                         |   494.612 |     1.5%
react/no-unused-prop-types           |   492.833 |     1.5%
react/no-direct-mutation-state       |   369.681 |     1.1%
import/no-extraneous-dependencies    |   312.265 |     0.9%
react/require-render-return          |   237.468 |     0.7%
react/jsx-no-target-blank            |   119.953 |     0.4%

4. без циклических и выключенный import/no-cycle:

Rule                                 | Time (ms) | Relative
:------------------------------------|----------:|--------:
@typescript-eslint/naming-convention |  9857.736 |    36.3%
prettier/prettier                    |  8042.446 |    29.7%
import/namespace                     |  4817.001 |    17.8%
react/no-unused-prop-types           |   550.794 |     2.0%
import/order                         |   496.272 |     1.8%
react/no-direct-mutation-state       |   395.921 |     1.5%
import/no-extraneous-dependencies    |   344.228 |     1.3%
react/require-render-return          |   248.769 |     0.9%
react/jsx-no-target-blank            |   132.625 |     0.5%
react/no-deprecated                  |   117.986 |     0.4%

Подумал что будет интересно.

[WbICHA]

Александр, оу, спасибо, реально полезно, я в целом как-то не задумывался, что у еслинта есть возможность замера времени работы рулов.)
Впрочем, стоит учитывать, что есть ещё достаточно много тяжёлых рулов и, поэтому, отключение одного конкретно этого может куда как слабее сказаться на результате, как в моём случае.

[WbICHA]

Александр, кстати, а зачем у тебя преттиер в еслинте включён? Сам же преттиер проверять файлы умеет.

[Александр]

WbICHA, проверять тайминг можно через TIMING=1 и команда, а Преттиер скорее для того, чтобы опр. правила работали - наверняка не скажу, т.к. конфиг с рабочего проекта наизусть не помню :)