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: Add relative-imports support #62

Open
3 tasks
azinit opened this issue Dec 30, 2021 · 4 comments
Open
3 tasks

LINT: Add relative-imports support #62

azinit opened this issue Dec 30, 2021 · 4 comments

Comments

@azinit
Copy link
Member

azinit commented Dec 30, 2021

Description

Should work for each rule

  • imports-order
  • layers-slices
  • public-api

Reference

// Pass
import { ... } from "entities/user";
import { ... } from "../entities/user";
import { ... } from "../../entities/user";
...
@azinit
Copy link
Member Author

azinit commented Dec 30, 2021

Относительные импорты - плохо, но кажется конфиг должен и для относительных импортов корректно работать 🤔

@Krakazybik
Copy link
Member

Krakazybik commented Jan 2, 2022

@azinit А какие с ними проблемы? Я вроде бы проверил данные кейсы, и вроде как работают. Можешь дать по точнее примеры, а то не совсем понимаю о чем речь идёт.

@azinit
Copy link
Member Author

azinit commented Jan 8, 2022

А какие с ними проблемы?

@Krakazybik См. выше)

"Относительные импорты - плохо, но кажется конфиг должен и для относительных импортов корректно работать 🤔"

Я вроде бы проверил данные кейсы, и вроде как работают.

Тогда надо в тесты добавить явно. Чтобы мы были уверены, что наши разные boundaries работают и для относительных импортов (а не только абсолютных, которые подразумевали в большинстве случаев)

Можешь дать по точнее примеры, а то не совсем понимаю о чем речь идёт.

Как пример для imports-order:

// Разные по вложенности импорты для проверки, 
// по факту же - они все равно все в src на одном уровне будут

// Fail
import { ... } from "../shared/button"; 
import { ... } from "../../entities/user";
import { ... } from "features/auth-form";

// Pass
import { ... } from "features/auth-form";
import { ... } from "../../entities/user";
import { ... } from "../shared/button";

Для layers-slices, public-api аналогично - чтобы в тестах учитывались и относительные импорты и сами правила работают как ожидаются

@azinit
Copy link
Member Author

azinit commented Jan 16, 2022

вот такое тоже отлавливать надо по идее

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants