Обзоры кода TFS - отображение обновленных файлов в ответ на комментарии


127

Мы начинаем использовать функцию проверки кода, встроенную в предварительную версию VS 2012 и VS 2013. Запросить обзор и добавить комментарии кажется довольно простым делом. Если кто-то добавляет комментарии с просьбой изменить код, то как запрашивающий вносит эти изменения и показывает их?

Итак, процесс будет происходить следующим образом:

  1. Человек 1 запрашивает проверку кода.
  2. Человек 2 добавляет комментарии и выбирает «Требуется работа».
  3. Человек 1 вносит необходимые изменения.

Как Человек 1 теперь показывает эти изменения Человеку 2? Вы можете добавлять комментарии и отправлять их, но файлы не меняются. Я предполагаю, что это файлы из набора изменений, созданного при запросе исходного обзора. Следует ли лицу 1 закрыть эту проверку и запросить повторную проверку?

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

Существует множество веб-сайтов, на которых показано, как использовать базовые функции обзоров кода, но есть ли какие-либо сайты, демонстрирующие лучшие практики? Например, кто должен устанавливать флажки рядом с файлами?


+1 за "есть ли сайты, демонстрирующие передовой опыт". Рейнджеры ALM обычно являются моими "ребятами" для более глубокого использования, но единственная информация, которую я могу найти, не отвечает на ваш вопрос. Я хотел бы увидеть некоторые пошаговые инструкции и сценарии использования Code Review.
DaveShaw

Ответы:


164

Итак, процесс будет происходить следующим образом:

  1. Человек 1 запрашивает проверку кода.
  2. Человек 2 добавляет комментарии и выбирает «Требуется работа».
  3. Человек 1 вносит необходимые изменения.
  4. Человек 1 Обновляет набор полок, связанный с проверкой кода.
  5. Человек 1 добавляет комментарии, чтобы продолжить обсуждение
  6. Повторяйте шаги 2–5 до подтверждения.

Вот шаги, необходимые для обновления набора полок, связанного с обзором.

  1. На панели «Обзор кода» выберите ссылку «просмотреть набор полок».
  2. На панели «Сведения о наборе полок» выделите и скопируйте имя набора полок.
  3. Перейдите на панель «Ожидающие изменения», нажмите «Полка» и вставьте имя набора полок.
  4. Нажмите кнопку Да в диалоговом окне проверки замены полочного набора.
  5. Теперь рецензент может видеть обновленные файлы, и обсуждение обзора может продолжаться.

Я включил несколько снимков экрана, поскольку считаю, что это помогает прояснить ситуацию.


1) На панели «Обзор кода» выберите ссылку «просмотреть набор полок», как показано здесь:

введите описание изображения здесь


2) На панели «Сведения о наборе полок» выделите и скопируйте имя набора полок, как показано здесь:

введите описание изображения здесь


3) Перейдите на панель «Ожидающие изменения», нажмите «Полка» и вставьте имя набора полок, например:

введите описание изображения здесь


4) Нажмите кнопку Да в диалоговом окне проверки замены полочного набора:

введите описание изображения здесь


7
Проблема с этим решением в том, что оно деструктивно, поскольку все комментарии сдуваются. Я думаю, что лучшим решением было бы создать новый обзор кода и установить старый как связанный элемент.
Bardia

15
Замена полочного набора не затрагивает комментарии, только файлы. Наша команда успешно использует этот метод, и мы не теряем никаких комментариев.
Чад Динхарт

10
Это работает, но если вы добавляете / удаляете строки при редактировании файла и заменяете набор полок, комментарии, которые были в этом файле, будут указывать на неправильные строки.
Чин

2
Я обычно не люблю повторяться, но точка @Cin выше делает этот ответ совершенно несостоятельным. Да, вы можете легко обновить свой набор полок, но при этом практически ВСЕ предыдущие комментарии обзора теперь указывают на НЕПРАВИЛЬНЫЙ текст. (Потому что, очевидно, комментарий к обзору прикреплен к абсолютному смещению от начала файла.)
Майкл Соренс,

7
Слишком ручной и не идеальный вариант. В TFS или Visual Studio должно быть что-то для сравнения изменений между последовательными проверками, чтобы проверяющий мог точно видеть изменения, внесенные с момента последней проверки. Ничего подобного не существует? Может быть, плагин или что-то в этом роде ...
Iúri dos Anjos

6

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


2

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

Недавно команда TFS начала этапы планирования решения очень старого запроса на голосовое сообщение пользователя, чтобы включить обновление Code Review, которое должно элегантно решить эту проблему, сделав итерации Code Reviews.


1
Отлично. Я тоже с нетерпением этого жду. Я бы приветствовал реальное решение. Мой пост - просто способ обойти проблему.
Чад Динхарт,

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

0

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

Этот рабочий процесс описан для обзоров на основе наборов изменений, но он также работает и для обзоров на полках.

  1. Создать задачу1
  2. Перед проверкой набора изменений 1 добавьте задачу 1 как связанный рабочий элемент
  3. Зарегистрируйте изменения с одним связанным рабочим элементом и запросите проверку этого набора изменений
  4. Создать задачу2
  5. Перед проверкой набора изменений 2 добавьте обе задачи как связанный рабочий элемент.
  6. Зарегистрируйте изменения с двумя связанными рабочими элементами и запросите проверку этого набора изменений

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

Используя наш сайт, вы подтверждаете, что прочитали и поняли нашу Политику в отношении файлов cookie и Политику конфиденциальности.
Licensed under cc by-sa 3.0 with attribution required.