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

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

chmnkh opened this issue · comments

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

минусы:

  • import Rofl нету автокомплита, и можно легко опечататься; import { Rofl } - есть автокомплит
  • export { default as Rofl } from - много буков; export * from - мало буков

плюсы:

  • удобнее экспортировать: не надо писать скобки и промежуточные переменные, если мы пишем хоки какие-то, например

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

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

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

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

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

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

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