Обзор кода перед регистрацией в TFS 2013

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

Есть ли какой-либо процесс, требующий пересмотра кода человеком, прежде чем он будет добавлен в реальный проект?

Я использую TFS 2013 с Visual Studio 2013.

ОТВЕТЫ

Ответ 1

TFS 2013 поддерживает это из коробки, и он очень прост в использовании.

Разработчик должен подать запрос на просмотр в проводнике команд:

  • Перейдите в Team Explorer
  • Откройте ожидающие изменения
  • В меню "Действия" выберите "Просмотр запроса"
  • Укажите рецензента и отправьте.

Пользователь (в вашем случае, менеджер) получит уведомление (если вы настроили параметры электронной почты в TFS), а также можете просмотреть запросы на просмотр в Team Explorer > My Work.

Рецензент может видеть все изменения, внесенные разработчиком, комментировать их и отправлять комментарии разработчику. Все это происходит внутри Visual Studio, и все комментарии хранятся в TFS и могут быть сообщены позже.

Кроме того, после завершения обзора рецензент может указать свое мнение об изменениях как "Looks Good", "With Comments" или "The Needs Work".

Убедитесь, что вы прочитали эту страницу в MSDN по этому вопросу: http://msdn.microsoft.com/en-us/library/hh474795.aspx

Вы также можете посмотреть это видео на канале 9: http://channel9.msdn.com/Series/Visual-Studio-2012-Premium-and-Ultimate-Overview/Visual-Studio-Ultimate-2012-Using-Code-Review-to-Improve-Quality

ИЗМЕНИТЬ

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

https://visualstudiogallery.msdn.microsoft.com/c476b708-77a8-4065-b9d0-919ab688f078

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

Если вам действительно нужно запретить какие-либо проверки, вы должны отказаться от разрешения на совершение действий для разработчиков и отправить их изменения менеджеру с помощью проверки кода (или набора полки). Затем менеджер может удалить изменения и зафиксировать после просмотра.