Слушайте, всё это правильно, кроме одного пункта... Создать дополнительный этап проверки внутри IDE. Речь о плагине для популярных сред разработки: проверять код можно будет прямо во время написания. Ревью станет быстрее и эффективнее — до централизованной проверки на сервере код уже будет избавлен от всех тривиальных ошибок.
Во-первых, вы отучаете разработчика думать. Во-вторых, ревьюеры не увидят (правильно понимаю ?) исправления ии-ревьюера при первичной проверки кода. А джунам (и не только) бы эти исправления было почитать интересно. А теперь перейдём к практике) Представьте, что в таблицу вашей БД разработчик добавил колонку с названием "created_updated_timestamp" и типом timestamp. Что сделает ваш ии-ревьюер? 1. Предложит убрать из имени "timestamp" 2. Предложит вместо одной колонки добавить 2 колонки: "created_at" и "updated_at" 3. Ничего не предложит.
Слушайте, всё это правильно, кроме одного пункта...
Создать дополнительный этап проверки внутри IDE. Речь о плагине для популярных сред разработки: проверять код можно будет прямо во время написания. Ревью станет быстрее и эффективнее — до централизованной проверки на сервере код уже будет избавлен от всех тривиальных ошибок.
Во-первых, вы отучаете разработчика думать. Во-вторых, ревьюеры не увидят (правильно понимаю ?) исправления ии-ревьюера при первичной проверки кода. А джунам (и не только) бы эти исправления было почитать интересно.
А теперь перейдём к практике) Представьте, что в таблицу вашей БД разработчик добавил колонку с названием "created_updated_timestamp" и типом timestamp. Что сделает ваш ии-ревьюер?
1. Предложит убрать из имени "timestamp"
2. Предложит вместо одной колонки добавить 2 колонки: "created_at" и "updated_at"
3. Ничего не предложит.