-
Notifications
You must be signed in to change notification settings - Fork 4
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
Comments
Относительные импорты - плохо, но кажется конфиг должен и для относительных импортов корректно работать 🤔 |
@azinit А какие с ними проблемы? Я вроде бы проверил данные кейсы, и вроде как работают. Можешь дать по точнее примеры, а то не совсем понимаю о чем речь идёт. |
@Krakazybik См. выше) "Относительные импорты - плохо, но кажется конфиг должен и для относительных импортов корректно работать 🤔"
Тогда надо в тесты добавить явно. Чтобы мы были уверены, что наши разные boundaries работают и для относительных импортов (а не только абсолютных, которые подразумевали в большинстве случаев)
Как пример для // Разные по вложенности импорты для проверки,
// по факту же - они все равно все в 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"; Для |
Description
Should work for each rule
imports-order
layers-slices
public-api
Reference
The text was updated successfully, but these errors were encountered: