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

The bemhtml lib can use without bem-tools #1

Open
blond opened this issue Aug 21, 2013 · 9 comments
Open

The bemhtml lib can use without bem-tools #1

blond opened this issue Aug 21, 2013 · 9 comments

Comments

@blond
Copy link

blond commented Aug 21, 2013

Проблема заключается в том, что bemhtml можно использовать вне bem-tools, например в enb. Но peerDependencies заставляет вытягивать его, когда этого совсем не хочется.

@SevInf
Copy link

SevInf commented Aug 22, 2013

Да, правильным решением было бы вынести lib/bemhtml в отдельный модуль. Поиск в npm среди всего прочего выдает https://github.com/indutny/bemhtml.js, очень похоже на старую версию lib/bemhtml. @indutny, возможно ли в твой репозиторий подтянуть изменения из bem-core и опубликовать новую версию?

@blond
Copy link
Author

blond commented Aug 22, 2013

@SevInf, почему бы не положить его в какой-нибудь bem/some_name репозиторий, и использовать его этой библиотекой?

@indutny, не подскажешь, почему lib/bemhtml не является частью bem-xjst?

@SevInf
Copy link

SevInf commented Aug 22, 2013

@andrewblond, вполне можно. Просто у @indutny есть уже опубликованный пакет с более старой версией этого кода, логичней было бы обновить его. Если это невозможно по каким-то причинам, сделаем новый пакет.

@blond
Copy link
Author

blond commented Aug 22, 2013

@SevInf, спасибо!

На сколько я понимаю название репозитория bemhtml.js будет не очень правильным, должно быть, что-то вроде bem-xjst.js, плюс когда репозиторий находится в bem о нём больше знают извне.

@indutny
Copy link

indutny commented Aug 22, 2013

Это не старая версия. Это bem-xjst, который я изначально разрабатывал у себя и под другим именем.

@tadatuta
Copy link

tadatuta commented Dec 7, 2013

@veged @indutny @arikon Еще один повод вынести lib/bemhtml — это тот факт, что оно используется в bem-bl@bem-xjst, bem-core и здесь тупым копипастом :(

Есть причины этого не сделать?

@arikon
Copy link

arikon commented Dec 7, 2013

@tadatuta Конечно, надо вынести

@veged
Copy link

veged commented Dec 9, 2013

@arikon @tadatuta, если я правильно понял @indutny, то сейчас это bem-xjst и нужно вместо копипаста везде использовать его

@tadatuta
Copy link

tadatuta commented Dec 9, 2013

сделал pr в bem-core: bem/bem-core#330

по-прежнему остается общий кусок, который будет копипаститься, только теперь на уровне технологии для сборки bemhtml (https://github.com/bem/bem-core/blob/feature/329%40v1/.bem/techs/bemhtml.js#L58-L75), но, наверное, это уже более-менее ок.

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

6 participants