Внедрение code style в существующий проект

[T-362]

Немного странный подход, правильный конечно, но ИМХО лучше с другой стороны. Наш пример — были взяты стандартные PSR темплейты от phpcs, phpmd и phpstorm, слегка дополнены персональными предпочтениями и были розданы разработчикам. Собственно и все, IDE автоматически форматирует и грязно ругается на попытки писать не по стандартам. Никто не накоммитит не по стандартам, если не может писать не по стандартам.

А вот рефакторинг старого кода это отдельная история.

[dvalid]

Использование анализатора на машинах разработчиков — правильное решение, но все же не обеспечивает 100% контроль. Как ни как, настройка инструмента и его корректное использование остается на совести каждого отдельного разработчика.
Централизованная система проверки дает больше гарантий, что новый код соответствует установленному формату. Думаю, наиболее эффективный подход — в сочетании обоих методов.

[Vorchun]

Мы думали проверять при заливке в SVN. Но это стопорит.
Потом ночью проверку, а утром тим лиду отчет за прошедший день. Тоже не пошло.
В итоге — настроены PHPStorm нужные правила. После итерации код прогоняется в общем хранилище и выделяется время на исправление.
Сначала было больно. Потом культура стала приходить в массы.

[Dreyk]

pre-commit хук точно также можно выключить

для проверки нужен pre-receive server-side hook. когда-то делал так с ПХП, работало нормально. проблемой было то, что стоит только тронуть один из старых файлов, и приходилось целиком его чистить

[Vorchun]

да, все правильно говорите )

[Cromathaar]

Наличие конвенции важнее самой конвенции. Разруха — она, как известно, в головах.