Как было устроено код-ревью на предыдущей работе
Python
Senior
Без компании
Как было устроено код-ревью на предыдущей работе
Ответы
На предыдущем месте код-ревью проводилось через GitHub/GitLab. Каждый пулл-реквест (PR) должен был:
- Иметь описание изменений
- Включать тесты (если это новая функциональность)
- Пройти CI/CD пайплайн
- Получить минимум 2 апрува от коллег
Ревьюеры проверяли:
- Соответствие код-стайлу проекта (flake8/black)
- Корректность архитектурных решений
- Наличие edge-case обработок
- Читаемость кода
Пример процесса:
```
# Ветка feature/new-endpoint
def process_data(data: list) -> dict:
if not isinstance(data, list): # Добавили проверку типа
raise ValueError("Expected list")
return {item['id']: item for item in data}
```
Комментарии в ревью выглядели так:
`⚠️ Нет обработки пустого списка` → добавляли `if not data: return {}`
Особенности:
- Время ревью не должно было превышать 24 часа
- Критику всегда сопровождали предложением решения
- Автоматические линтеры сокращали 30% ручной работы