search

gulp-bem / gulp-bem-xjst

DEPRECATED repository, moved to https://github.com/bem/gulp-bem/tree/master/packages/gulp-bem-xjst
MIT License
6 stars 3 forks source link

addAttrs is not defined #74

Closed maksimenka closed 7 years ago

maksimenka commented 7 years ago

При сборке c помощью Gulp block('button').mod('type', 'link')( tag()('a'), addAttrs()({ id: 'test', name: 'test' }) );

qfox commented 7 years ago

Дело в версии bem-xjst, в поставке какая-то 6.х, кажется

maksimenka commented 7 years ago

А возможно его обновить, очень много функций не работает? Или есть альтернатива использованию bem-xjst при сборке галпом?

qfox commented 7 years ago

Возможно, но нет библиотек, которые совместимы с 8.х, в которой есть addAttrs. Вы отдельно используете bem-xjst?

maksimenka commented 7 years ago

За основу брал: https://github.com/bem/project-stub Но меня интересует только bem-xjst, т.е. я убрал bem-core и bem-components. Сборка в целом работает, просто начал описывать логику, и часть функций не работает которые описаны в документации.

qfox commented 7 years ago

Думаю что делать. Можно, конечно, топорно выпускать версии, но это надо делать вместе с релизами bem-xjst

cc @miripiruni

maksimenka commented 7 years ago

Мне по душе ваш шаблонизатор, но проблема в кадрах разбирающихся в i-bem и тд. Поэтому мы хотим использовать bem-xjst с фреймворком ангуляр. И возможно я не один такой.

miripiruni commented 7 years ago

@zxqfox а что тут думать? Нужно, чтобы ментейнер gulp-bem-xjst выпустил версии поддерживающие bem-xjst 7.x и 8.x

miripiruni commented 7 years ago

Можно настроить greenkeeper. Oh, wait… 39 пулреквестов… )

qfox commented 7 years ago

@miripiruni ну тогда нужно чтобы мейнтейнером этого пакета был ты)

miripiruni commented 7 years ago

@zxqfox я не могу поддерживать все метапакеты, которые есть в интернете. Ну то есть могу, но такие вещи нужно решать greenkeeper-ом.

miripiruni commented 7 years ago

@zxqfox не рассматриваешь вариант пустить поддерживать эти пакеты тех людей, которые действительно ими пользуются?

miripiruni commented 7 years ago

Я как бы намекаю на некоторых участников этого треда…

maksimenka commented 7 years ago

Как я понимаю @tadatuta, допилит ;)

tadatuta commented 7 years ago

Я с неделю назад заслал https://github.com/gulp-bem/gulp-bem-xjst/pull/69 Если мейнтейнера де-факто нет, то готов влить и выпустить версию.

@zxqfox, @blond даете добро?

miripiruni commented 7 years ago

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

tadatuta commented 7 years ago

@miripiruni done

miripiruni commented 7 years ago

@tadatuta я всё ещё не могу управлять пулреквестами…

maksimenka commented 7 years ago

Когда ориентировочно ожидать мержа новой версии?

miripiruni commented 7 years ago

@Maksimenka сегодня.

maksimenka commented 7 years ago

Ок, спасибо.

miripiruni commented 7 years ago

Выпустил

gulp-bem-xjst@3.0.0 c зависимостью bem-xjst ^8.3.1 и gulp-bem-xjst@2.0.0 c зависимостью bem-xjst ^7.5.0