search

feature-sliced / eslint-config

🍰 Lint feature-sliced concepts by existing eslint plugins
https://npmjs.com/@feature-sliced/eslint-config
MIT License
119 stars 4 forks source link

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

Open azinit opened 2 years ago

azinit commented 2 years ago

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 commented 2 years ago

Конфиг:

'@feature-sliced/eslint-config/rules/import-order',
    '@feature-sliced/eslint-config/rules/layers-slices',
    '@feature-sliced/eslint-config/rules/public-api/lite',
azinit commented 2 years ago

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

Krakazybik commented 2 years ago

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

noveogroup-amorgunov commented 1 year ago

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

azinit commented 1 year ago

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

illright commented 1 year ago

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