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

Добавьте jscs с принятым у вас стайлгайдом(если он есть) #625

Open
ghost opened this issue Mar 28, 2015 · 5 comments

Comments

@ghost
Copy link

ghost commented Mar 28, 2015

Сложно читать код, в котором то '==', то '===', итп.

Да я думаю, что и вам самим удобнее будет. И для контрибьютеров проще. Думал PR сделать, а в каком стиле вы пишете не понятно.

@qfox
Copy link
Member

qfox commented Mar 29, 2015

@tadatuta что думаешь на счет PR?

JSCS конфиг могу запилить, мне не сложно ;-)

@qfox qfox mentioned this issue Apr 14, 2015
23 tasks
@tadatuta
Copy link
Member

@zxqfox JFYI, сейчас ребята заканчивают работу по перелопачивания существующих issues в bem-bl, внесут текущие правки и после этого в bem-bl наступит фриз, а еще через какое-то время, впрочем, наверное, достаточное большое — полностью прекратится любая поддержка.
пора переходить на bem-core.

@qfox
Copy link
Member

qfox commented Apr 15, 2015

А как модули подключить? Тот PR отребейзить?

@qfox
Copy link
Member

qfox commented Apr 15, 2015

@telegraphinho Не проверял на совместимость, но пробелы починит и остальное расскажет (в корне проекта):

npm i jscs jscs-bem
echo '{ "plugins": ["jscs-bem"], "preset": "bem" }' > .jscsrc
jscs -x blocks-*

@tadatuta
Copy link
Member

А как модули подключить?

Там так и не пришли к консенсусу, так что пока только ждать решения или подключать на своем уровне.

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