Поделиться через


Получение обратной связи с помощью запросов на вытягивание

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

Проверки кода

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

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

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

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

Отправка отзывов

Качественные проверки начинаются с качественных отзывов. Ключевые факторы, определяющие качественные отзывы в запросе на вытягивание:

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

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

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

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

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

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

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

Подробнее

GitHub содержит обширную документацию по предложениям изменений в работе с запросами на вытягивание.

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