Skip to content

Conversation

@ijakparov
Copy link

It alows to develop projects for touch and desktop using different bundles and to create bundles for different pages

## Installation

```sh
npm i -g enb
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

зачем ставить enb глобально?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хороший вопрос, тогда его нужно в зависимости добавить, у меня в Win 10 ошибку без этого выбивало.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

он же есть в зависимостях: https://github.com/bem/bem-express/blob/master/package.json#L50

@tadatuta
Copy link
Member

общие вещи:
0. сейчас сборка падает

  1. хочется, чтобы пулл-реквесты вносили максимально атомарные изменения. например, в рамках данного pr-а нет смысла обновлять nodemon (он, кстати, и так по мягким зависимостям поставится свежий) или ставить enb глобально (это в принципе звучит спорно, ведь каждый проект может ожидать свою собственную версию enb).

  2. изменение структуры директорий — это вкусовщина. я предлагаю пока оставить исходную.

  3. не должно быть изменений, которые позже откатываются (как например с console.log)

  4. не должно быть коммитов с названием вроде «Merge branch 'express-multipage'»

@ijakparov
Copy link
Author

общие вещи:
0. сейчас сборка падает

  1. хочется, чтобы пулл-реквесты вносили максимально атомарные изменения. например, в рамках данного pr-а нет смысла обновлять nodemon (он, кстати, и так по мягким зависимостям поставится свежий) или ставить enb глобально (это в принципе звучит спорно, ведь каждый проект может ожидать свою собственную версию enb).
  2. изменение структуры директорий — это вкусовщина. я предлагаю пока оставить исходную.
  3. не должно быть изменений, которые позже откатываются (как например с console.log)
  4. не должно быть коммитов с названием вроде «Merge branch 'express-multipage'»

Понял, буду работать в этом направлении

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

Successfully merging this pull request may close these issues.

2 participants