search

fullstack-development / react-redux-starter-kit

Modular starter kit for React+Redux+React Router projects.
https://demo.fullstack-development.com/
MIT License
91 stars 13 forks source link

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

Closed sk1e closed 4 years ago

sk1e commented 4 years ago

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

prodderman commented 4 years ago

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

sk1e commented 4 years ago

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

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

in19farkt commented 4 years ago

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

prodderman commented 4 years ago

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

prodderman commented 4 years ago

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

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

sk1e commented 4 years ago

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

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

chmnkh commented 4 years ago

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

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

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

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

sk1e commented 4 years ago

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

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

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

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

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

in19farkt commented 4 years ago

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

chmnkh commented 4 years ago

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