alfa-laboratory / arui-presets-lint

A set of linting presets for arui-feather-based projects
14 stars 4 forks source link

Feat/add ts no shadow #122

Closed PoytaL closed 3 years ago

PoytaL commented 3 years ago

Изменение no-shadow проверки

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

При обновлении на проектах версии TypeScript до 4+ возникает необходимость повышения версии typescript-eslint. В typescript-eslint 4.0+ возникает баг на объявлениях enum.

Снимок экрана 2021-02-26 в 11 34 51

Изменения выключают стандартную проверку no-shadow, заменяя ее на проверку @typescript-eslint/no-shadow из typescript-eslint 4.0+, что позволяет обойти возникающий баг.

PoytaL commented 3 years ago

Стандартное правило eslint 'no-use-before-define' отрабатывает не корректно при проверках

https://github.com/alfa-laboratory/arui-presets-lint/actions/runs/610415084

При наличии правила из @typescript-eslint базовое правило из eslint следует отключить

https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-use-before-define.md#disallow-the-use-of-variables-before-they-are-defined-no-use-before-define

IBelyaev commented 3 years ago

:tada: This PR is included in version 5.7.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: