eslint-config icon indicating copy to clipboard operation
eslint-config copied to clipboard

LINT: (Beta) [public-api] Falsy positive linting for inner submodules

Open azinit opened this issue 3 years ago • 6 comments

Problem: Даже с лайтконфигом - ложно триггерится на импорты внутренностей внутри слайса Rules: public-api/lite Version: 0.1.0-beta.5 Reference: @pzyryanov1995

Уже была эта проблема, видно не пофиксилось лайт конфигом 🤷‍♂️ https://github.com/feature-sliced/eslint-config/discussions/75#discussioncomment-2055274

image

azinit avatar Feb 07 '22 10:02 azinit

Конфиг:

'@feature-sliced/eslint-config/rules/import-order',
    '@feature-sliced/eslint-config/rules/layers-slices',
    '@feature-sliced/eslint-config/rules/public-api/lite',

azinit avatar Feb 07 '22 10:02 azinit

Возможно трабла в том, что у нас такие ./foo/bar импорты только для shared в тесткейсах встречаются, а для других слоев не работает 🤔 (но надо чекнуть)

azinit avatar Feb 07 '22 10:02 azinit

Действительно lite не отрабатывает :thinking:

Krakazybik avatar Feb 13 '22 19:02 Krakazybik

Как сейчас решаете сейчас эту проблему?

noveogroup-amorgunov avatar Apr 24 '23 20:04 noveogroup-amorgunov

@noveogroup-amorgunov Сейчас планируем чуть перетрясти реализацию public-api правила, чтоб пофиксить этот и остальные баги public-api

azinit avatar Apr 26 '23 05:04 azinit

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

illright avatar May 27 '23 14:05 illright