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

WARNING! vanilla.js browser.js node.js #459

Closed
ilyar opened this issue Mar 20, 2014 · 7 comments
Closed

WARNING! vanilla.js browser.js node.js #459

ilyar opened this issue Mar 20, 2014 · 7 comments

Comments

@ilyar
Copy link
Contributor

ilyar commented Mar 20, 2014

Технологии browser.js и node.js основаны на vanilla.js которая основана на базовой технологии 'js', ели в реализации vanilla.js нет exports.API_VER = 2; то при создании файла технологии мы получаем варнинг.

@arikon Это фича которая подразумевает переопределение этих технологий на уровне проекта или бага?

@narqo
Copy link
Member

narqo commented Mar 21, 2014

Это фича которая подразумевает переопределение этих технологий на уровне проекта или бага?

Можешь уточнить, ты на уровне проекта описываешь свою технологию vanilla.js, которая не наследуется ни от кого? В этом случае нужно явно указывать, что vanilla.js реализует вторую версию API-технологий.

Либо я не понял вопроса.

По факту, у нас vanilla.js наследуется от технологии v2/js-i — это прописано в конфигах уровней

@ilyar
Copy link
Contributor Author

ilyar commented Mar 21, 2014

@narqo спасибо за пояснение, смотрю в конфигах уровней project-stub, при создании блоков получаем:

WARNING! Tech module "browser.js" at C:\Users\test\project\libs\bem-core.bem\techs\browser.js.js is using API V1.

Что надо сделать что бы технология использовалась в V2? Надо указать описании уровня проекта 'js': 'v2/js-i', или как то иначе?

@narqo
Copy link
Member

narqo commented Mar 21, 2014

Что надо сделать что бы технология использовалась в V2? Надо указать описании уровня проекта 'js': 'v2/js-i', или как то иначе?

Да, чтобы пропал ворнинг, достаточно добавить в конфиг уровня блоков декларацию для технологии js. Пример.

Дело в том, что правки в project-stub вносились во времена когда bem-tools еще не смотрел конфиги уровней блоков во время интроспекции. Будет сдорово, если сможешь поправить это вместе с bem/project-stub#39

\cc @tadatuta

@ilyar
Copy link
Contributor Author

ilyar commented Mar 21, 2014

@narqo еще уточни имеет знание порядок описания технологий?

@ilyar
Copy link
Contributor Author

ilyar commented Mar 21, 2014

ок, дополню bem/project-stub#39

@narqo
Copy link
Member

narqo commented Mar 21, 2014

@narqo еще уточни имеет знание порядок описания технологий?

Нет, это же хеш — JS не гарантирует порядок ключей в нем ;)

@ilyar
Copy link
Contributor Author

ilyar commented Mar 21, 2014

👍 спасибо

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

2 participants