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/types/guards.ts #163

Open sk1e opened 4 years ago

sk1e commented 4 years ago

Выражения с typeof и так являются гвардами. Функция лучше оператора, но оператор лучше функции с необходимостью её импорта и захламления кодобазы, imo

Znack commented 4 years ago

Ну вот функция лучше оператора, это да. И в целом isString довольно удобная штка, даже при наличии typeof value === 'string', я сам такой функцией не раз пользовался уже (когда передать надо refinement куда-нибудь). Но тут единственная причина удаления — в других местах это не используется уже, видимо по историческим причинам осталось

Safr commented 4 years ago

type guards кастомные лучше выносить в отдельную папку, поэтому выпиливать не нужно, это же переизспользуемые штуки для принципа dry