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

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

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

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

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

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

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


tfs c# tfs-code-review visual-studio-2012
person John    schedule 19.08.2013    source источник
comment
+1 за то, что есть сайты с передовой практикой. Рейнджеры ALM обычно мои ребята, готовые к более глубокому использованию, но единственная информация, которую я могу найти, не отвечает на ваш вопрос. Я хотел бы увидеть некоторые пошаговые инструкции и сценарии использования Code Review.   -  person DaveShaw    schedule 20.08.2013

Ответы (4)


arrow_upward
174
arrow_downward

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

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

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

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

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

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

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

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

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

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

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

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

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

person Chad Dienhart    schedule 14.01.2014
comment
Проблема с этим решением в том, что оно деструктивно, поскольку все комментарии сдуваются. Я думаю, что лучшим решением было бы создать новый обзор кода и установить старый как связанный элемент. - person Bardia; 21.04.2014
comment
Замена полочного набора не затрагивает комментарии только к файлам. Наша команда успешно использует этот метод, и мы не теряем никаких комментариев. - person Chad Dienhart; 22.04.2014
comment
На данный момент это лучший ответ. Я поделюсь этим с нашей командой. - person John; 08.05.2014
comment
Это работает, но если вы добавляете / удаляете строки при редактировании файла и заменяете набор полок, комментарии, которые были в этом файле, будут указывать на неправильные строки. - person Chin; 29.10.2014
comment
У меня это не работает. После того, как запрашивающая сторона заменяет набор полок, проверяющий получает сообщение об ошибке при попытке открыть рецензию кода. В нем говорится, что набор полок не может быть найден или у меня нет разрешения. - person nthpixel; 03.07.2015
comment
Какую версию TFS и VS вы используете? Когда вы заменяете набор полок, вы видите диалоговое окно окончательной проверки? - person Chad Dienhart; 03.07.2015
comment
Как ты это понял !! Ракета! Это должно быть функцией, но этот обходной путь потрясет пригороды. - person granadaCoder; 24.09.2015
comment
Обычно я не люблю повторяться, но указанная выше точка @Cin делает этот ответ совершенно несостоятельным. Да, вы можете легко обновить свой набор полок, но при этом практически ВСЕ предыдущие комментарии обзора теперь указывают на НЕПРАВИЛЬНЫЙ текст. (Потому что, очевидно, комментарий обзора прикреплен к абсолютному смещению от начала файла.) - person Michael Sorens; 10.03.2016
comment
@MichaelSorens Это обходной путь, позволяющий рецензенту увидеть изменения, внесенные человеком 1 после комментариев рецензента. Если человеку 1 необходимо внести существенные изменения по сравнению с исходной рецензией, возможно, лучше закрыть оригинал и открыть новую рецензию. Я просто пытаюсь помочь обойти недостаток инструмента проверки кода TFS. Это не универсальное решение, но во многих случаях оно хорошо работает. - person Chad Dienhart; 10.03.2016
comment
Я ценю ваш комментарий, Чад, и полностью понимаю, что это обходной путь. Я использую обзор кода TFS каждый день в своей работе, но это очень больное место. (Не ваше предложение, а тот факт, что TFS не поддерживает пересмотр кода! :-) Я рад, что это обходное решение полезно во многих случаях для вас, но, к сожалению, в моей среде оно просто бесполезно вообще. :-( - person Michael Sorens; 10.03.2016
comment
Слишком ручной и не идеальный вариант. В TFS или Visual Studio должно быть что-то для сравнения изменений между последовательными проверками, чтобы проверяющий мог точно видеть изменения, внесенные с момента последней проверки. Ничего подобного не существует? Может быть, плагин или что-то в этом роде ... - person Iúri dos Anjos; 14.02.2017
comment
@ Iúri dos Anjos - Я согласен, это не идеал - это обходной путь, пока Microsoft не исправит его. У вас есть плагин, который подходит для этого? - person Chad Dienhart; 02.08.2017
comment
@ChadDienhart ничего не знаю. Думаю, Microsoft этого не исправит. Придется использовать / перейти на Git. - person Iúri dos Anjos; 02.08.2017
comment
@ IúridosAnjos есть несколько инструментов для сравнения наборов полок, но они в лучшем случае неуклюжие - person USER_8675309; 12.03.2018
comment
ПОЖАЛУЙСТА, перед этим посоветуйтесь с рецензентом. Он сохраняет все предыдущие комментарии, но все места в коде разные, и результат может быть невероятно трудным для чтения и понимания (например, AzDO PR, когда вы не выбираете конкретный номер обновления). И если вы сделаете это, вы потеряете исходный набор полок и никогда не сможете выровнять код и комментарии. Просто отправьте новый отзыв. Рецензенты в любом случае должны просматривать каждый отзыв, как если бы он был новым. - person StingyJack; 21.06.2020

arrow_upward
6
arrow_downward

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

person Paddy    schedule 20.08.2013

arrow_upward
3
arrow_downward

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

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

person julealgon    schedule 29.05.2015
comment
Большой. Я тоже с нетерпением этого жду. Я бы приветствовал реальное решение. Мой пост - просто способ обойти проблему. - person Chad Dienhart; 24.10.2015
comment
Надеюсь, это не будет похоже на запросы на извлечение лазурного DevOps. Это кошмар для спама. - person StingyJack; 05.12.2019
comment
Этот экземпляр UserVoice больше не доступен. - person ruffin; 12.08.2020
comment
Спасибо за предупреждение @ruffin. Обновил ответ с новой ссылкой. Проблема была перенесена из UserVoice на новый сайт. Вы можете дважды проверить, что это та же проблема, по URL-адресу, указанному на новой странице в разделе «Миграция с». - person julealgon; 12.08.2020

arrow_upward
0
arrow_downward

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

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

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

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

person Ville Pelho    schedule 08.10.2013