search

bem / bem-express

BEM project-stub with BEMTREE and express
43 stars 19 forks source link

Multipage express project for a few types of devices #157

Open ijakparov opened 5 years ago

ijakparov commented 5 years ago

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

tadatuta commented 5 years ago

общие вещи:

  1. сейчас сборка падает

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

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

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

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

ijakparov commented 5 years ago

общие вещи:

  1. сейчас сборка падает

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

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

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

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

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