Психологическая безопасность, доверие, общие ценности команды
Культура важнее процессов. Без доверия и психологической безопасности code review превращается в бюрократию.
Психологическая безопасность — атмосфера, где разработчики не боятся:
| Здоровая культура | Токсичная культура |
|---|---|
| «Хорошее замечание, спасибо!» | «Ты опять это сломал» |
| «Я не подумал об этом» | «Это очевидно же» |
| «Давай обсудим синхронно» | Спор на 50 комментариев |
| Ошибки обсуждаются открыто | Ошибки скрываются |
Для тимлида:
✅ Признавайте свои ошибки публично
✅ Благодарите за найденные баги в review
✅ Пресекайте личные атаки немедленно
✅ Поощряйте вопросы от junior разработчиков
Для рецензента:
✅ Начинайте с положительного: «Спасибо за PR! Несколько замечаний:»
✅ Используйте вопросы: «Что если попробовать...?»
✅ Признавайте, когда не правы: «Ты прав, это лучше»
Для автора:
✅ Отвечайте на все комментарии
✅ Благодарите за review
✅ Задавайте уточняющие вопросы
Низкое доверие:
Высокое доверие:
1. Качество кода — коллективная ответственность
2. Критика кода ≠ критика личности
3. Обучение важнее «быть правым»
4. Скорость важна, но не в ущерб качеству
5. Помощь коллегам — приоритет
Не работает:
Работает:
| Ритуал | Цель | Частота |
|---|---|---|
| Калибровка review | Обсудить, что считается хорошим review | Раз в месяц |
| Разбор инцидентов | Учиться на ошибках без обвинений | После инцидента |
| Tech talk | Делиться знаниями о кодовой базе | Раз в 2 недели |
| Pair review | Совместное review для обучения | По необходимости |
Цель: договориться, что считается хорошим замечанием.
Формат:
Результат: консистентные ожидания от review.
«Этот код недостаточно хорош, перепиши»
*без конкретных замечаний*
Решение: конкретные замечания с объяснением.
«Я проверил 50 PR за день!»
*каждый за 2 минуты*
Решение: лимит на количество PR в день, фокус на качестве.
«Ты допустил баг, теперь месяц будешь делать review всем»
Решение: review — ответственность всех, не наказание.
*PR от junior висит 3 дня без review*
Решение: SLA для всех, менторство как приоритет.
Ключевая мысль: Культура строится ежедневно. Каждое замечание в review — кирпичик в фундаменте доверия или токсичности.
Вопросы ещё не добавлены
Вопросы для этой подтемы ещё не добавлены.