Культура работы с Git: код-ревью и пуллреквесты

Продолжаем серию статей «Культура работы с Git» от Юлии Волковой. В предыдущих постах рассказали про именование веток и о правильных сообщениях к коммитам.

Код-ревью — один из важнейших аспектов работы с Git. Это не только про код, это ещё и про взаимодействие внутри команды. Речь не только о том, чтобы отлавливать баги или спорить о стайлгайдах.

Если подойти к процессу осознанно, он принесёт команде и каждому участнику очень много пользы.

Код-ревью помогает:
🎯быть в курсе контекста по разным задачам в проекте;
🎯совместно принимать решения: от стайлгайдов и выбора библиотек до архитектуры;
🎯улучшать качество кода и вырабатывать более оптимальные решения;
🎯ещё на этапе разработки нового функционала оценить масштабируемость, поддерживаемость решения, учесть возможные риски и corner cases;
🎯обратить внимание на необходимость переиспользования каких-то компонентов или утилит;
🎯быстрее знакомить новых сотрудников с проектом;
🎯обучаться и улучшать свои навыки - как хард-скиллы, так и софт-скиллы.

Как видите, код-ревью — это ещё и про коммуникацию, общение и взаимодействие внутри команды. А ещё про ценности: например, обмен знаниями, обучение, взаимопомощь, стремление сделать лучше.

🔖Посмотрите небольшой доклад «Код-ревью с уважением» Ангелины Купцовой с митапа SPB Frontend, чтобы узнать о теме больше. И советую почитать статью «Что такое код-ревью» от Доки.

⤵️ В следующих постах я поделюсь рекомендациями, которые помогут сделать работу в пуллреквесте и ревью немного более полезным и комфортным.