а как тогда core team форматирует фреймворк?

Никак. На pull request'ах всё контролируется.

Просто не тащат плохо оформленное.

руками контролируется?

Ну конечно, там же куча человек смотрит pull request.

https://github.com/yiisoft/yii2-coding-standards/blob/master/Yii2/ruleset.xml

Вот тут вообще ничего особенного и нет.

Весь файл в TODO.

у Yii2 есть некоторый отступления по стандарту. Где их найти?

у Yii2 есть некоторый отступления по стандарту. Где их найти?

Это какие? Второй вроде полностью следует тому, что в PSR.

я хочу с CI сделать проверку и при разработке автоматически контролировать стандарт.

ну ты же там сам писал, что оно отступает от неокторых правил

Эм, я вроде про Symfony писал.

Там Фабьен решил делать не: "str" . "str" А так: "str"."str"

Ну там реально мелочи.

Всё это настройками настраивается. Смысла в том репозитории точно нет.

например private $_ranges = [];

подчеркивание приватных переменных

это не по PSR

Ты тоже в своих проектах нижнее подчёркивание используешь у свойств?

настройки - это понятно. Я хочу подогнать свой код точно так, как он отформатирован в офф репо ии.

да, тоже использую, так как это принято в ии

Я честно говоря не совсем понимаю, зачем они это подчёркивание у private сделали.

Какая была мотивация в этом.

иногда реально удобно

в чем удобство, реально?

я не буду холливарить. Мой подход в именованию методов и магия ии дает свои плоды.

Ну тогда бери те правила, хорошо. =) Только они для PHP CodeSniffer, а не для PHP CS Fixer.

Для PHP CS Fixer вроде тоже можно свой rule set написать.

вот в том то и дело)

phpcs настроил. Дальше как настроить fixer?

где взять полны список правил по которым работает phpcs ?

https://github.com/FriendsOfPHP/PHP-CS-Fixer

Так вот тут же.

Там в README прямо список.

это я понимаю. Я хочу список правил для Yii2 а не полный список.

Не годится каждое правило фиксера перебирать руками и сравнивать "а юзается оно в phpcs" или нет

я на это неделю убью ((

Как вариант воспользоваться PSR1, PSR2, Symfony правилами, а потом переопределить то, что в Yii иначе.

https://github.com/FriendsOfPHP/PHP-CS-Fixer

во, то есть Yii2 правила выстроены на основе набора PSR1, PSR2, Symfony и некоторые моментики из ruleset.xml переиначены. Так?

https://github.com/2amigos/yii2-usuario/blob/master/.php_cs.dist

Вот примерно.

То, что для Yii2 подходит.

Но я лично private свойства не пишу со знака подчёркивания, поэтому это не добавлял в этот файлик.

Какая была мотивация в этом.

Сразу видишь визуальное отличие публичной переменной и приватной

Но ащета, есть еще protected, что насчет него

Проще всего юзать psr2, оно и в штормах настраивается в 1 клик, даже можно обойтись без своего xml-файла :)

Проще всего юзать psr2, оно и в штормах настраивается в 1 клик, даже можно обойтись без своего xml-файла :)

Но как до этого говорили, есть некоторые недочеты

ну да, понятно) я пишу так как это экология ии2. В проектах на других фреймах я придерживаюсь экологии тех фреймов :)

типо длины строки

типо длины строки

120 символов много?

я думал больше

но вообще, 120 - это критическая длина, лучше 80

там мильйон недочетов) так что код контролируетя так 1. PhpStorm (PSR1\PSR2) 2. PHP SC Fixer 3. Php Code Sniffer

я когда в компанию пришел, и мне наглядно показали почему только на шторм полагаться нельзя) с тех пор сниффер юзаю

там мильйон недочетов) так что код контролируетя так 1. PhpStorm (PSR1\PSR2) 2. PHP SC Fixer 3. Php Code Sniffer

на гит хук повестить проверку коммитов, не давать запушить, разве что в мастер разрешить (ворнинг показывать) для супербыстрого фикса

на гит хук повестить проверку коммитов, не давать запушить, разве что в мастер разрешить (ворнинг показывать) для супербыстрого фикса

перед тем как что-то автоматиизировать, надо сперва настроить инструменты) сейчас об этом разговор - юзать или нет)

https://github.com/2amigos/yii2-usuario/blob/master/.php_cs.dist

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

Пока что не понял чего именно не хватает, если в phpcs через шторм указать psr2

только длина строки?

Пока что не понял чего именно не хватает, если в phpcs через шторм указать psr2

писать долго, попробуй просто свой проект прогнать сниффером)

писать долго, попробуй просто свой проект прогнать сниффером)

чего его прогонять, использую каждый день со сниффером, ни разу без него не юзал

а, так мы же вчера говорили вроде. Я тебе пример приводил даже.

ERROR: S client not available

разве нет?

мне? не помню такого

сек, повторю сейчас

мне? не помню такого

https://gyazo.com/e93751388a6c63970589c8901400ae82 тут видно, что автоформатирование штора отформатировало код не по PSR и приходиться руками подправлять) Это только один маленький пункт, который наглядно показывает, что шторм знает о длинне строк и пытается ее исправить исправляя код... но длает это как знает, а не как требует стандарт проекта.

я когда в компанию пришел, и тоже был удивлен чем им штормовское форматирование не нравиться. И мне тогда наглядно показали в коде что не так он делает) вопросы отпали, поставил сниффер

https://gyazo.com/e93751388a6c63970589c8901400ae82 тут видно, что автоформатирование штора отформатировало код не по PSR и приходиться руками подправлять) Это только один маленький пункт, который наглядно показывает, что шторм знает о длинне строк и пытается ее исправить исправляя код... но длает это как знает, а не как требует стандарт проекта.

Так я же не про этого говорю...а про то, что свой xml-файл можно не делать, так как для phpcs в шторме можно выбрать psr2. Правка всего этого дела - вопрос второй.

Так я же не про этого говорю...а про то, что свой xml-файл можно не делать, так как для phpcs в шторме можно выбрать psr2. Правка всего этого дела - вопрос второй.

а, ты про настройку валидации в инспекторе для сниффера? Да, можно просто выбрать PSR2. Но все же если есть файлик конфига для фрейма, можно кастомный настроить) хуже не будет

внимание, вопрос

зачем трейтам protected свойства

у них что, есть наследование?

?

ну могут пригодится протектед методы(я правда не знаю где)

у них что, есть наследование?

внезапно есть

как это вообще выглядит

http://andy-carter.com/blog/overriding-extending-a-php-trait-method

как это вообще выглядит

а, типо, use внутри трейта?

и что, protected при таком раскладе работает как надо?

@Enapiuz

и что, protected при таком раскладе работает как надо?

да работает. только что проверил

класс

а блин, трейты не умеют extends же :)

а блин, трейты не умеют extends же :)

а я был прав =(

не использую трейты :) точней есть 1 случай у меня, но оно особо не считается

один раз не альпинист)

крута в ubuntu git на русском)

та говорят это не круто

крута в ubuntu git на русском)

эм, а что там перевели хоть ?

ну всё что в консоли было, git commit, git status

help. очень удобно кстати

Так это не в убунте, а просто если русская локаль выставлена