Мы начинаем использовать функцию проверки кода, встроенную в предварительную версию VS 2012 и VS 2013. Запросить обзор и добавить комментарии кажется довольно простым делом. Если кто-то добавляет комментарии с просьбой изменить код, то как запрашивающий вносит эти изменения и показывает их?
Итак, процесс будет происходить следующим образом:
- Человек 1 запрашивает проверку кода.
- Человек 2 добавляет комментарии и выбирает «Требуется работа».
- Человек 1 вносит необходимые изменения.
Как Человек 1 теперь показывает эти изменения Человеку 2? Вы можете добавлять комментарии и отправлять их, но файлы не меняются. Я предполагаю, что это файлы из набора изменений, созданного при запросе исходного обзора. Следует ли лицу 1 закрыть эту проверку и запросить повторную проверку?
Создание второго обзора не кажется оптимальным, потому что вы теряете историю разговора о том, почему вы вносили изменения.
Существует множество веб-сайтов, на которых показано, как использовать базовые функции обзоров кода, но есть ли какие-либо сайты, демонстрирующие лучшие практики? Например, кто должен устанавливать флажки рядом с файлами?
Ответы:
Итак, процесс будет происходить следующим образом:
Вот шаги, необходимые для обновления набора полок, связанного с обзором.
Я включил несколько снимков экрана, поскольку считаю, что это помогает прояснить ситуацию.
1) На панели «Обзор кода» выберите ссылку «просмотреть набор полок», как показано здесь:
2) На панели «Сведения о наборе полок» выделите и скопируйте имя набора полок, как показано здесь:
3) Перейдите на панель «Ожидающие изменения», нажмите «Полка» и вставьте имя набора полок, например:
4) Нажмите кнопку Да в диалоговом окне проверки замены полочного набора:
источник
Я считаю, что правильная процедура предназначена для Человека 1, чтобы внести изменения и запросить повторную проверку. Когда ваш код нуждается в доработке, это означает, что вы будете его изменять, поэтому вы захотите использовать старую версию для сравнения. У вас все еще есть старый отзыв в истории после его закрытия, если вы хотите просмотреть комментарии. В настоящее время мы оптимизируем наш процесс проверки кода на моем рабочем месте.
источник
Я знаю, что это старый вопрос, но он все еще не поддерживается, как указано в других плакатах. Решение, предложенное chad, будет работать для одних вещей, но будет иметь странное поведение для других.
Недавно команда TFS начала этапы планирования решения очень старого запроса на голосовое сообщение пользователя, чтобы включить обновление Code Review, которое должно элегантно решить эту проблему, сделав итерации Code Reviews.
источник
Вы должны сделать это с двумя разными отзывами. Но есть также способ сохранить историю со вторым обзором. Все, что вам нужно, это задачи.
Этот рабочий процесс описан для обзоров на основе наборов изменений, но он также работает и для обзоров на полках.
Теперь во втором запросе проверки рецензент может искать связанные задачи, и если рецензент ищет задачу1, он / она видит набор изменений1 и запрос на проверку с комментариями. Так что вы не потеряете историю разговоров.
источник