fullstack-development / react-redux-starter-kit

Modular starter kit for React+Redux+React Router projects.

Home Page:https://demo.fullstack-development.com/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Выпилить shared/helpers[replaceObjectKeys, replaceObjectValues]

sk1e opened this issue · comments

Легко заменяются на R.mapObjIndexed. Захламляют кодовую базу

Серегей, ты сам то пробовал это выпилить с помощью R.mapObjIndexed? @sk1e

Серегей, ты сам то пробовал это выпилить с помощью R.mapObjIndexed? @sk1e

я так понял replaceObjectKeys нельзя выпилить этим, да? Думаю нужно вот это использовать тогда

А оно сейчас где-то используется в демке? Если нет, то смело выпииваем, не нужно ни на что заменять.

все эти хэлперы используются

я так понял replaceObjectKeys нельзя выпилить этим, да? Думаю нужно вот это использовать тогда

да это подойдет, но ради этого либу тянуть?

да это подойдет, но ради этого либу тянуть?

что тебя смущает?

да это подойдет, но ради этого либу тянуть?

что тебя смущает?

не знаю, что Степана смущает, но меня смущает, что мы держим в зависимостях библиотеку ради одной функции из нее

ну и что, что тришейкнется, держать целую зависимость в пэкэйдж джейсоне, когда можно просто 1 файл на 5 строк написать - это какой-то прикол вообще, я даже не понимаю, че мы тут спорим

да это подойдет, но ради этого либу тянуть?

что тебя смущает?

не знаю, что Степана смущает, но меня смущает, что мы держим в зависимостях библиотеку ради одной функции из нее

ну и что, что тришейкнется, держать целую зависимость в пэкэйдж джейсоне, когда можно просто 1 файл на 5 строк написать - это какой-то прикол вообще, я даже не понимаю, че мы тут спорим

Ты во-первых пишешь - тратишь время; во-вторых держишь модулем, заставляя всех запоминать что именно такая вот херня у нас есть в хелперах и если что используйте её, противопоставляя цельной методологии - все общие алгоритмы брать из ramda/ramda-adjunct. В третьих какая разница сколько ты берёшь из этой библиотеки? Когда тебе понадобится вторая функция из этой библиотеки, что ты будешь делать? В четвёртых ты так говоришь как будто эту зависимость в package.json ты держишь собственными руками и это довольно тяжёло. Ну лежит она там, в чём проблема конкретно? npm'у тяжелее работать будет?

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

короче ок, я на самом деле не могу рационально объяснить, почему мне не нравится, поэтому давайте воткну