react-redux-starter-kit icon indicating copy to clipboard operation
react-redux-starter-kit copied to clipboard

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

Open chmnkh opened this issue 5 years ago • 6 comments

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

минусы:

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

плюсы:

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

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

chmnkh avatar Oct 18 '19 03:10 chmnkh

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

krashaen avatar Oct 18 '19 03:10 krashaen

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

chmnkh avatar Oct 18 '19 04:10 chmnkh

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

sk1e avatar Oct 18 '19 05:10 sk1e

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

Znack avatar Oct 18 '19 08:10 Znack

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

kinda-neat avatar Nov 28 '19 06:11 kinda-neat

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

chmnkh avatar Nov 28 '19 06:11 chmnkh