Skip to content
This repository has been archived by the owner on Jan 13, 2018. It is now read-only.

Add ability to specify level prototype as a name (close #367, BEM-949) #435

Merged
merged 4 commits into from
Sep 13, 2013

Conversation

scf2k
Copy link
Contributor

@scf2k scf2k commented Aug 21, 2013

No description provided.

@scf2k
Copy link
Contributor Author

scf2k commented Sep 10, 2013

@narqo @arikon вливаем это или нет?

@narqo
Copy link
Contributor

narqo commented Sep 10, 2013

Это ведь про то, что можно будет писать

// desktop.blocks/.bem/level.js
exports.baseLevelPath = 'blocks';

и оно само поймет, что я хочу?

👾

@scf2k
Copy link
Contributor Author

scf2k commented Sep 10, 2013

Про то. Будет искать такой уровень в bem/lib/levels

@@ -4,14 +4,36 @@ var PATH = require('./path'),
FS = require('fs'),
INHERIT = require('inherit'),
createTech = require('./tech').createTech,
util = require('util'),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

По нашему code style нужно UTIL

@arikon
Copy link
Contributor

arikon commented Sep 11, 2013

@narqo Не совсем. Оно про то, что уровни из bem-tools можно будет использовать по имени, директивой baseLevelName. Без необходимость указывать полный путь к модулю во внутренностях bem-tools.

// .bem/level.js
exports.baseLevelName = 'project';

@arikon
Copy link
Contributor

arikon commented Sep 12, 2013

LGTM. Портируй в support/0.6.x ещё.

scf2k added a commit that referenced this pull request Sep 13, 2013
Add ability to specify level prototype as a name (close #367, BEM-949)
@scf2k scf2k merged commit 9a7a488 into release-1.0.0 Sep 13, 2013
@scf2k scf2k deleted the BEM-949 branch September 13, 2013 12:34
scf2k added a commit that referenced this pull request Sep 16, 2013
Add ability to specify level prototype as a name (close #367, BEM-949)
scf2k added a commit that referenced this pull request Sep 16, 2013
backport: Merge pull request #435 from bem/BEM-949
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants