Доброе утро. Подскажите кто каким подходом пользуется при организации ревью чужого кода. 1) Каким правилам следовать, чтобы это не превращалось в батл. Просто договориться по стилю, например psr2, по всяким не ооп делам - это вот эти запахи кода Фаулера, ну а по ООП уже сложнее. 2) Чисто организационно сам процесс заключается в: - исполнитель шлет merge request - и в комментах к нему происходит общение - исполнитель исправляет, жмет комменту шефа "resolved", и пушит в эту ветку новые коммиты - и общение уже в новых коммитах продолжается
а зачем тут батл... если делается код ревью значит тот кто его делает - знает лучше как делать чем исполнитель)) раз сказали что надо сделать по другому - надо переделать)) а так да - все правильно))
А смотрю не нетбинс, ни пхпсторм не заменят своими psr1/2 средствами форматирования array() на [] и кучу других хотелок типа else if на elseif . Это так у всех или я недонастроил?
прежде всего контролируй себя сам. можно настроить проверку на pre-commit https://medium.com/@setkyarwalar/setting-up-phpcs-on-laravel-908bccb82db