eslint-config icon indicating copy to clipboard operation
eslint-config copied to clipboard

LINT-45: Split layers-slices-boundaries

Open Krakazybik opened this issue 3 years ago • 10 comments

Description

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

Reference

#45

Checks

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

Krakazybik avatar Dec 19 '21 09:12 Krakazybik

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

azinit avatar Dec 19 '21 16:12 azinit

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

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

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

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

azinit avatar Dec 19 '21 17:12 azinit

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

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

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

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

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

Krakazybik avatar Dec 19 '21 17:12 Krakazybik

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

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

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

azinit avatar Dec 19 '21 17:12 azinit

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

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

azinit avatar Dec 20 '21 16:12 azinit

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

Krakazybik avatar Dec 23 '21 10:12 Krakazybik

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

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

azinit avatar Dec 23 '21 19:12 azinit

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

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

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

Krakazybik avatar Dec 23 '21 19:12 Krakazybik

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

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

azinit avatar Dec 24 '21 20:12 azinit

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

azinit avatar Dec 24 '21 20:12 azinit