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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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