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

Отказаться от дефолтных экспортов? #152

Open chmnkh opened 4 years ago

chmnkh commented 4 years ago

зачем использовать дефолтные экспорты?

минусы:

плюсы:

минусы сильно побеждают, ящитаю

krashaen commented 4 years ago

если примем это требование, оно будет вводится только на новых проектах или как?)

chmnkh commented 4 years ago

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

sk1e commented 4 years ago

Такое лучше оформлять ПРом на стандарты, где ты приведёшь все доводы против дефолтных экспортов и примеры по их замене с учётом всех случаев. А потом уже создавать ишью или сразу ПР на приведение к стандартам в стартер ките.

Znack commented 4 years ago

В целом я согласен, что дефолтных лучше избегать. И я согласен, что лучше начать со стандартов. А еще лучше, надо явно прописать правило eslint-а, которое будет это чекать :)

kinda-neat commented 4 years ago

это походу можно пометить как ready for work: запилить пр в стандарты

chmnkh commented 4 years ago

надо договориться о тонкостях всяких тогда, типа че делать, если надо вернуть компоненту под хоками и тд