feature-sliced / eslint-config

🍰 Lint feature-sliced concepts by existing eslint plugins

https://npmjs.com/@feature-sliced/eslint-config

MIT License

120 stars 5 forks source link

LINT-45: Split layers-slices-boundaries #46

Open Krakazybik opened 2 years ago

Krakazybik commented 2 years ago

Description

Разбиты на отдельные конфиги slice и layers boundaries. Пока сохранен slices-and-layers-boundaries (потом нужно будет написать мерж конфигов) Добавлены тесты для slices-and-layers-boundaries.

Reference

45 Checks

[x] Description added
[x] Self-reviewed
[x] CI pass

azinit commented 2 years ago

Спс за крутецкое оформление PR! Описание прям в темплейт вынести можно ✊

azinit commented 2 years ago

@Krakazybik Я кста не просто так в отдельную таску вынес сплиттинг layers и boundaries))

Т.к. на данном этапе это скорее больше проблем создает, и PR тоже разбухает

+ Я вспомнил, что мы рили вродь как договаривались, что забиваем на данный момент на сплиттинг

А делаем это отдельно по #45 задаче (ее щас и не закроешь еще как минимум из-за того что не все требования выполнены) 🤷‍♂️

Krakazybik commented 2 years ago

@Krakazybik Я кста не просто так в отдельную таску вынес сплиттинг layers и boundaries))

Т.к. на данном этапе это скорее больше проблем создает, и PR тоже разбухает

Я вспомнил, что мы рили вродь как договаривались, что забиваем на данный момент на сплиттинг

А делаем это отдельно по #45 задаче (ее щас и не закроешь еще как минимум из-за того что не все требования выполнены) man_shrugging

окай =) оно просто уже поспличено было, значит будет как #45ый, 22ой то я проверил, там вроде бы всё?

azinit commented 2 years ago

окай =) оно просто уже поспличено было, значит будет как #45ый, 22ой то я проверил, там вроде бы всё?

Да, по 22-му тогда только в ридмиху инфу добавить бы как частично подключать (даже с layers-slices-boundaries единым)

А потом отдельный PR на преттир+форматирование
И этот PR уже на #45

azinit commented 2 years ago

Кста если PR этот драфтовым хотел сделать - лучше рили добавить WIP: префикс в тайтле

А то уже ревьювить думал))

Krakazybik commented 2 years ago

Вливать сначало МЕНЯ!

azinit commented 2 years ago

Вливать сначало МЕНЯ!

Так это же вообще про 0.2.0 история вродь как, не?)

Krakazybik commented 2 years ago

Вливать сначало МЕНЯ!

Так это же вообще про 0.2.0 история вродь как, не?)

Окай =( снова конфликты разруливать :D

azinit commented 2 years ago

Да у меня честно говоря пока в целом сомнения, насколько стоит сплитить щас layers и slices, учитывая единственно возможную реализацию

Я мб даже бы этот PR на паузу поставил 🤔

azinit commented 2 years ago

(+ учитывая что с public-api щас те же самые проблемы появились)