Здравствуйте, peer, Вы писали:
P>Текущее состояние системы очень плохое и хотим внедрить код ревью для начала.
Почему Вы считаете, что состояние системы "плохое". В чем это выражается?
P>Но есть сомнения что текущий код не позволит соблюдать правила кода.
Не надо создавать слишком сложные правила кода. Начните с малого — хотя бы просто приучите людей к ревью.
Для этого нужно ревью сделать обязательным перед заливкой в мастер-ветку. Это само по себе уже даст прибавку в качестве кода.
Требования к коду можно добавлять/усиливать по мере улучшения кодовой базы.
P>Я вижу что покрытие системы тестами на ключевых операциях, потом рефактор ключевых проблем в коде и потом внедрение код ревью.
Нужно поощрять писать хоть какие-то осмысленные юнит-тесты. Это можно включить в правила ревью.
Но нужно учитывать, что при плохом качестве кода иногда невозможно протестировать вообще.
На существующий код можно (и нужно) написать функциональные тесты.
Это позволит отрефакторить основные проблемные участки.
