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

new README file #73

Merged
merged 41 commits into from
Oct 11, 2016
Merged

new README file #73

merged 41 commits into from
Oct 11, 2016

Conversation

godfreyd
Copy link
Contributor

No description provided.


[![NPM Status][npm-img]][npm]
Copy link
Member

Choose a reason for hiding this comment

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

Hey, everybody loves such badges. And they are very useful while searching new npm module.

Copy link
Member

Choose a reason for hiding this comment

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

+1


[npm]: https://www.npmjs.org/package/bem-walk
[npm-img]: https://img.shields.io/npm/v/bem-walk.svg
Инструмент обходит файловую структуру БЭМ-проекта и возвращает информацию о найденных файлах.
Copy link
Member

Choose a reason for hiding this comment

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

Что такое «файловая структура»?

bem-walk
========

[![NPM Status][npm-img]][npm]
Copy link
Contributor

Choose a reason for hiding this comment

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

Надо бы вернуть беджики

[david]: https://david-dm.org/bem-sdk/bem-walk
[david-img]: http://img.shields.io/david/bem-sdk/bem-walk.svg?style=flat

Инструмент позволяет обойти файловую систему БЭМ-проекта и получить следующую информацию о найденных файлах:
Copy link
Member

Choose a reason for hiding this comment

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

давайте во всех пакетах всегда держать первый абзац с основным описанием пакета перед бейджами? @miripiruni показывал, что если класть их сразу после заголовка, это негативно сказывается на поиск пакета в npm

Copy link
Contributor

@qfox qfox Aug 25, 2016

Choose a reason for hiding this comment

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

А можно посмотреть, где он это показывал? Вообще, я первый раз слышу, что через npm можно что-то вменяемое найти.

Copy link
Member

Choose a reason for hiding this comment

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

Когда рассказывал про bem/bem-xjst@abab495 вбивал bem-xjst в саджест npm и там вместо описания пакета всплывала HTML-разметка бейджиков.

Copy link
Contributor

@qfox qfox Aug 25, 2016

Choose a reason for hiding this comment

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

Убедительно. 👍 Только там очень мало места и надо компактно, умело владея языком.
В полтвиттера.

@@ -1,5 +1,11 @@
# bem-walk

Инструмент позволяет обойти файловую систему БЭМ-проекта и получить следующую информацию о найденных файлах:
Copy link
Contributor

Choose a reason for hiding this comment

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

Не не, как раз такое не очень хорошо делать.

Лучше написать кратко про назначение модуля, типа: «Штука для обхода файловой системы».

Может быть поменять на:

Инструмент для обхода файловой системы БЭМ-проекта.

Получает следующую информацию о найденных файлах:
…

[david]: https://david-dm.org/bem-sdk/bem-walk
[david-img]: http://img.shields.io/david/bem-sdk/bem-walk.svg?style=flat

Bem-walk позволяет получить следующую информацию о найденных файлах:
Copy link
Member

Choose a reason for hiding this comment

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

Давай название пакета/инструмента всегда писать так: bem-walk.

Copy link
Member

Choose a reason for hiding this comment

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

Может упростим?

Позволяет получить информацию о найденных файлах:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Возвращает информацию о найденных файлах: Как тебе?

Copy link
Member

Choose a reason for hiding this comment

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

годно!

})
.pipe(through2.obj(function(file, enc, callback) {

if (fs.statSync(file.path).isFile()) {
Copy link
Member

Choose a reason for hiding this comment

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

Этот код надо обернуть в try/catch, и на ошибку вызывать callback(err)

@blond blond merged commit 296b04b into master Oct 11, 2016
@blond blond deleted the godfreyd-bem-walk branch October 11, 2016 16:00
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.

None yet

5 participants