-
Notifications
You must be signed in to change notification settings - Fork 22
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
Давайте удалим этот документ и ссылки на него? #220
Conversation
The `node.getLevelNamingScheme` and the `node.setLevelNamingScheme` methods will be removed in `enb@2.0.0`.
Got rid of scheme builders
Redundant code of deps
Remove deprecated techs
Remove deprecated options
The `mock-fs` does not support `require` for Node.js 4. Because of this test failed with error: ``` Cannot find module 'path/to/file.js ```
Support Node.js 4
**Example of new format:** ``` { button__text: [ { entity: { block: ‘button’, elem: ‘text’ }, tech: ‘css’, path: ‘path/to/file.ext’, level: ‘path/to/level’ }, /* ... */ ], /* ... */ } ``` With this introspection format to easily get a list of files named entity. **What has been done?** * Use `bem-walk` to scan levels. * Use `bem-naming` to get id of BEM entity. * Add `BundleIntrospection` class to work with introspection of levels for one bundle. * Rewrite `levels` tech with `build-flow`. * Use promises with `node.buildState` to avoid scanning the same levels multiple times. * Don't read files for directories (example i18n), it should do `files` tech.
Now the `bem-walk` is used to scan.
Exactly the same benchmarks in `bem-walk` repository
Introspection Format
API moved to bottom of list
В чём смысл этого документа? Он дублирует https://ru.bem.info/methodology/filesystem/ и не привносит ничего нового. Я предлагаю удалить этот документ и ссылку на него из README.md
какой-то очень странный дифф получился |
Дифф для того, чтобы написать коммент. Я предлагаю удалить этот документ полностью. |
так и удалил бы его в PR-е ;) |
Давай удалим, но оставим ссылку на документ из bem.info. |
@blond я не понимаю, что ты предлагаешь. Я предлагаю добавить ссылку на https://ru.bem.info/methodology/filesystem/ в README.md Так? |
Да, всё так. |
Closed in favor of #252 |
В чём смысл этого документа? Он дублирует https://ru.bem.info/methodology/filesystem/ и не привносит ничего нового.
Я предлагаю удалить этот документ и ссылку на него из README.md
//cc @blond @tadatuta