Onboarding, менторство, распространение знаний о кодовой базе
Code review — это не только проверка кода, но и мощный инструмент обучения и распространения знаний о кодовой базе.
Что даёт review:
Как помочь junior:
✅ Развёрнутые объяснения: не просто «исправь», а «почему»
✅ Ссылки на документацию и примеры
✅ Предлагайте pair review для сложных PR
✅ Хвалите за хорошие решения
Пример комментария:
«Хорошее использование list comprehension!
Для обработки ошибок рекомендую посмотреть на пример в
`services/user_service.py:45` — мы используем там контекстный
менеджер для транзакций.
Документация: https://docs.python.org/3/library/contextlib.html»
Что даёт review:
Неделя 1:
Day 1-2: Чтение документации, настройка окружения
Day 3: Первый PR (документация, мелкий багфикс)
Day 4: Review от ментора с подробными комментариями
Day 5: Исправление, обсуждение
Неделя 2-4:
- Постепенное усложнение задач
- Ментор назначает 2 рецензента для важных PR
- Pair review раз в неделю
- Ретроспектива через месяц
□ Выделен ментор для нового разработчика
□ Первые 5 PR ревьювятся ментором
□ Проведён pair review хотя бы один раз
□ Новый разработчик понимает процесс review
□ Есть список ресурсов для обучения
□ Запланирована ретроспектива через месяц
Проблема:
Команда A не знает, что делает Команда B
→ Дублирование функциональности
→ Несовместимые решения
→ «Это чёрный ящик, никто не знает как работает»
Решение:
✅ Назначать рецензентов из других команд
✅ Проводить open review для архитектурных изменений
✅ Вести changelog крупных изменений
| Формат | Частота | Цель |
|---|---|---|
| Tech talk | Раз в 2 недели | Глубокое погружение в тему |
| Show & tell | Раз в месяц | Демонстрация новых фич |
| Post-mortem | После инцидента | Учимся на ошибках |
| Code walkthrough | По необходимости | Разбор сложного кода |
Socratic method:
Вместо: «Используй здесь select_related»
Спросить: «Какой будет производительность этого запроса
при 1000 записей? Как можно оптимизировать?»
Show by example:
«Вот пример похожего кода в `services/order_service.py`:
[ссылка на файл]
Можно использовать аналогичный подход.»
Progressive feedback:
Неделя 1: Подробные комментарии на каждое замечание
Неделя 2-4: Комментарии только на важные вещи
Месяц 2+: Только критичные замечания, больше доверия
Для ментора:
«Я заметил, что ты часто спрашиваешь про обработку ошибок.
Давай выделим час на этой неделе и разберём эту тему глубже?»
От менти:
«Спасибо за подробные комментарии! Особенно помогли ссылки
на примеры в кодовой базе. Можно ли ещё добавить примеры
для транзакций?»
| Тип знания | Где хранить |
|---|---|
| Архитектура | ADR в репозитории |
| Процессы | Wiki (Notion, Confluence) |
| Примеры кода | Код в репозитории + комментарии |
| Решения из review | Комментарии в PR (поиск) |
| Best practices | Документация команды |
✅ Ссылки на PR в документации
✅ Теги для PR: #architecture, #security, #performance
✅ Changelog крупных изменений
✅ Запись tech talk с транскриптом
Ключевая мысль: Code review — инвестиция в рост команды. Тратьте время на объяснения сегодня, чтобы получить самостоятельных разработчиков завтра.
Вопросы ещё не добавлены
Вопросы для этой подтемы ещё не добавлены.