Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

webpack 2, Immutable.js и semicolons #1

Open
dlebedynskyi opened this issue Aug 30, 2016 · 7 comments
Open

webpack 2, Immutable.js и semicolons #1

dlebedynskyi opened this issue Aug 30, 2016 · 7 comments

Comments

@dlebedynskyi
Copy link

dlebedynskyi commented Aug 30, 2016

  1. На сервере - Node.js и Express.
    Koa
    See https:/lostpebble/koa-mobx-react-starter
  2. CSS modules и isomorphic-style-loader для автоматической генерации Critical CSS при Server-side Rendering.
    +1
  3. Redux для взаимодействия внутри приложения
    Redux + Immutable.js + Recompose + Reselect
    also redux-immutablejs to wire stuff together
  4. Модульное тестирование через AVA и Enzyme.
    yap

webpack 1.x для сборки. Или всё-таки webpack 2?
webpack 2 sound better now due to tree shaking

ESLint и eslint-config-airbnb с небольшим изменением - не использовать точку с запятой.
use semi-column ;

@DenisIzmaylov
Copy link
Contributor

DenisIzmaylov commented Aug 31, 2016

Точка с запятой - это и горячий хороший момент для дискуссий. Если откинуть фактор удобно/неудобно и привычно/непривычно, то:

  • Аргумент "за" отсутствие точки с запятой:
    • Быстрее писать код
    • Легче читабельность за счёт уменьшения "шума"
  • Аргумент против:
    • Есть несколько кейсов, когда это может вызвать ошибку

@DenisIzmaylov
Copy link
Contributor

Раньше использовали koa, это видно в open-parse. Однако у Express есть отличная поддержка вендоров и это чаще важнее небольших удобств и красоты кода. Пример поддержки:

@DenisIzmaylov
Copy link
Contributor

DenisIzmaylov commented Aug 31, 2016

В каких ситуациях Immutable.js может себя оправдать даже не смотря на увеличение кодовой базы?

@DenisIzmaylov
Copy link
Contributor

webpack 2 sound better now due to tree shaking

Были недавно аргументы в React.js Telegram Group, где webpack 2 ещё совсем beta и рождает проблемы. Разве это не так на данный момент?

@DenisIzmaylov DenisIzmaylov changed the title discuss webpack 2, Immutable.js и semicolons Aug 31, 2016
@DenisIzmaylov
Copy link
Contributor

@dlebedynskyi Если не возражаете, переименовал тему для репрезентативности.

@dlebedynskyi
Copy link
Author

dlebedynskyi commented Sep 1, 2016

@DenisIzmaylov
immutables спасают на больших store. главная особенность в том, что состояние всегда будет новой ref при изменении свойств или вложенных объектов.
В то же время только обновленные объекты получают new ref.
дальше - добавьте pure components with fast ref check for props change = profit.
в целом получаеться что ui - это оптимизированная функци состояния с гарантией того, что состояние объекта не будет изменено без изменения ссылки.

Были недавно аргументы в React.js Telegram Group, где webpack 2 ещё совсем beta и рождает проблемы. Разве это не так на данный момент?

это будет починено. а стек не на месяц.

@dlebedynskyi
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants