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

Introduce BEMTREE support #9

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

tadatuta
Copy link
Member

enb-bemxjst updated to ^8.6.7

@tadatuta
Copy link
Member Author

cc @miripiruni @blond

@levonet
Copy link

levonet commented Apr 27, 2017

У меня сразу встречное предложение: каждый коммит с апом версии enb-bemxjst протегать соответствующей мажорной версией. Так как мажорные версии bem-xjst от версии к версии взрывают многое.

@levonet levonet mentioned this pull request Apr 27, 2017
@miripiruni
Copy link
Member

TypeError: LRU: key must be a string or number. Almost certainly a bug! undefined
    at typeCheckKey (/home/travis/build/enb/enb-bemxjst-i18n/node_modules/bower/node_modules/lru-cache/lib/lru-cache.js:20:19)
    at LRUCache.get (/home/travis/build/enb/enb-bemxjst-i18n/node_modules/bower/node_modules/lru-cache/lib/lru-cache.js:230:3)
    at /home/travis/build/enb/enb-bemxjst-i18n/node_modules/bower/lib/core/resolvers/GitRemoteResolver.js:122:70
    at _fulfilled (/home/travis/build/enb/enb-bemxjst-i18n/node_modules/bower/node_modules/q/q.js:854:54)
    at self.promiseDispatch.done (/home/travis/build/enb/enb-bemxjst-i18n/node_modules/bower/node_modules/q/q.js:883:30)
    at Promise.promise.promiseDispatch (/home/travis/build/enb/enb-bemxjst-i18n/node_modules/bower/node_modules/q/q.js:816:13)
    at /home/travis/build/enb/enb-bemxjst-i18n/node_modules/bower/node_modules/q/q.js:877:14
    at runSingle (/home/travis/build/enb/enb-bemxjst-i18n/node_modules/bower/node_modules/q/q.js:137:13)
    at flush (/home/travis/build/enb/enb-bemxjst-i18n/node_modules/bower/node_modules/q/q.js:125:13)
    at process._tickCallback (node.js:458:13)
bower invalid-meta  bem-core is missing "main" entry in bower.json
bower resolved      https://github.com/bem/bem-core.git#2.8.0
bower install       bem-core#2.8.0
bem-core#2.8.0 fixtures/bem-core
module.js:340
    throw err;
          ^
Error: Cannot find module 'enb-bemxjst/node_modules/bemhtml-compat/node_modules/ometajs'

@tadatuta
Copy link
Member Author

@levonet, у меня есть несколько аргументов против:

  1. До сих пор это был мой личный форк от 0.x версии, так что никаких гарантий в принципе быть не могло.
  2. С точки зрения API этого пакета ничего не менялось.
  3. Для пользователей переходы между версиями bem-xjst должны быть дешевы благодаря автомиграции, а внезапно начать поддерживать старье не вижу смысла, как минимум пока не будут найдены живые пользователи, в этом заинтересованные.
  4. Главный аргумент: заниматься этим совершенно некому.

@tadatuta
Copy link
Member Author

@miripiruni это ожидаемое поведение, мы обсуждали — тесты были написаны в доисторические времена и с тех пор их никто не трогал. Починить их быстро я не смог, а чинить долго у меня мотивации нет.

@levonet
Copy link

levonet commented May 14, 2017

@tadatuta

  1. С точки зрения API этого пакета ничего не менялось.

enb-bemxjst-i18n вытягивает в зависимостях соответствующий bem-xjst, который и будет использоваться. Поскольку мой микропроектик использовал твой форк, Переключение форка с 7 на 8 у меня всё дико взорвало из-за escapeContent: true by default. Я с этим справился.

Но кажется, для консистентности, стоит хотяб начать назначать тот же мажор, что и у bem-xjst.

@Realetive
Copy link

Bump!

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

Successfully merging this pull request may close these issues.

4 participants