Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
azinit opened this issue Feb 7, 2022 · 6 comments
Open
Labels
bug Something isn't working

Comments

@azinit
Copy link
Member

azinit commented Feb 7, 2022

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

Уже была эта проблема, видно не пофиксилось лайт конфигом 🤷‍♂️
#75 (reply in thread)

image

@azinit
Copy link
Member Author

azinit commented Feb 7, 2022

Конфиг:

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

@azinit
Copy link
Member Author

azinit commented Feb 7, 2022

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

@Krakazybik
Copy link
Member

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

@noveogroup-amorgunov
Copy link

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

@azinit
Copy link
Member Author

azinit commented Apr 26, 2023

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

@azinit azinit added the bug Something isn't working label May 22, 2023
@illright
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants