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

Remove 'docs/bem-project.md' file #222

Merged
merged 1 commit into from
May 17, 2016
Merged

Conversation

vithar
Copy link
Contributor

@vithar vithar commented May 16, 2016

No description provided.

@@ -38,7 +38,8 @@ $ npm install --save-dev enb-bem-techs
Документация
------------

* [Как устроены БЭМ-проекты](docs/bem-project.md)
Об организации БЭМ-проектов в файловой системе читайте в разделе [методология](https://ru.bem.info/methodology/filesystem/) на сайте [bem.info](https://ru.bem.info/).
Copy link
Member

Choose a reason for hiding this comment

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

Может показаться, что ниже подразделы.

Давай добавим что-то вроде, «Документация пакета enb-bem-techs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Выше написано просто «Установка». Мы находимся внутри документа enb-bem-techs. Я не понимаю, зачем добавлять «пакета enb-bem-techs»

Copy link
Member

Choose a reason for hiding this comment

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

Об организации проектов в файловой системе читайте в БЭМ-методологии.

  • Сборка бандла

Легко подумать, что это подпункты про «Об организации проектов в файловой».

Я за любой вариант, который пояснит, что это не так ;)

@vithar
Copy link
Contributor Author

vithar commented May 17, 2016

Так?

@blond
Copy link
Member

blond commented May 17, 2016

👍

@blond blond merged commit a758bb0 into enb:master May 17, 2016
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

Successfully merging this pull request may close these issues.

None yet

2 participants