Получение отзывов с пул-реквестами

Пулл-реквесты поддерживают рассмотрение и объединение кода в едином процессе совместной работы. После добавления функции или исправления ошибок разработчик создает pull request, чтобы начать процесс объединения изменений в основную ветку. Затем другие члены команды получают возможность проверить и утвердить код до завершения. Используйте запросы на вытягивание для проверки работы и получения ранних отзывов об изменениях. Но нет никаких обязательств по слиянию изменений. Владелец может в любое время отклонить пулл-реквест.

Проверьте код

Проверка кода, выполненная в рамках пулл-реквеста, предназначена не только для поиска явных ошибок; для этого существуют тесты. Хороший обзор кода перехватывает менее очевидные проблемы, которые могут привести к дорогостоящим проблемам позже.

Проверки кода помогают защитить команду от плохих слияний и сборок с ошибками, которые подрывают производительность команды. Обзоры выявляют проблемы до слияния, защищая важные ветки от нежелательных изменений.

Проверки кода также поощряют и укрепляют совместную работу и обмен данными между разработчиками. И команда получает четкую историю всех изменений, внесенных между главной ветвью и функциональными ветками.

Обмен опытом и распространение стратегий по решению проблем с помощью широкого спектра рецензентов на проверках кода. Распространение навыков и знаний делает команду сильнее и устойчивее.

Предоставление отличного отзыва

Качественные проверки начинаются с качественных отзывов. Основы отличного фидбэка в пулл-реквесте:

• Пусть подходящие люди проверят запрос на вытягивание.
• Убедитесь, что рецензенты знают, что делает код.
• Предоставление практических, конструктивных отзывов.
• Своевременно отвечать на комментарии.

При назначении обозревателей к запросу на вытягивание убедитесь, что выбрали подходящих специалистов. Рецензенты должны знать, как работает код, но также включают разработчиков, работающих в других областях, чтобы они могли поделиться своими идеями.

Предоставьте четкое описание изменений и предоставьте сборку кода с исправлением или компонентом, работающим в нем. Рецензенты должны предпринять усилия для предоставления отзывов об изменениях, с которыми они не согласны. Определите проблему и предоставьте конкретные предложения о том, что можно сделать по-другому. Эта обратная связь имеет четкое намерение и легко воспринимается владельцем pull request.

** Владелец пулл-реквеста должен отвечать на комментарии, принимать предложения или объяснить, почему он отказывается их применять. Некоторые предложения хороши, но могут выходить за рамки пулреквеста. Выполните эти предложения и создайте новые рабочие элементы и ветви функций отдельно от запроса на вытягивание, чтобы внести эти изменения.

Защита ветвей с помощью политик

В репозитории есть несколько критически важных ветвей, на которые команды полагаются, что они всегда будут в хорошем состоянии, таких как ветвь main. Команды могут требовать pull requests для внесения изменений на этих ветвях с такими платформами, как GitHub и Azure DevOps. Разработчики, вносящие изменения непосредственно в защищенные ветви, столкнутся с отклонением своих изменений.

Добавьте дополнительные условия для pull request, чтобы обеспечить более высокий уровень качества кода в ключевых ветках. Чистая сборка объединенного кода и утверждение от нескольких рецензентов являются дополнительными требованиями, часто применяемыми для защиты ключевых веток.

Подробнее

GitHub содержит обширную документацию о том, как предложить изменения в вашей работе с помощью pull requests.

Дополнительные сведения о предоставлении отличных отзывов в проверках кода и использовании шаблонов pull request для обеспечения рекомендациями ваших рецензентов. Azure DevOps также предлагает удобный интерфейс запроса на вытягивание, который легко использовать и масштабировать по мере необходимости.