Code review и git flow

Код junior-разработчика читают люди, а не только процессор. Ревью и дисциплина веток превращают GitHub в командную культуру, а не свалку коммитов.

Цикл типовой задачи

1. создайте ветку от main: git checkout -b feat/payment;
2. делайте маленькие осмысленные коммиты (одна задача — один коммит);
3. пушьте и откройте Pull Request с описанием «что и зачем»;
4. ревью: комментарии, правки, апрув;
5. merge (CI зелёный) и удаление ветки.

Что смотреть в ревью

• логика и баги: верный ли результат, ожидаемые ли крайние случаи;
• безопасность: данные из ввода не должны попадать в SQL без валидации;
• читаемость: понятны ли имена, разбита ли задача на части;
• протестированность: есть ли тесты или хотя бы объяснение, как проверить.

Конфликты

git fetch origin
git rebase origin/main   # или git merge origin/main
# git разрешит руками, затем:
git add .
git rebase --continue