Відгуки про код 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) Натисніть кнопку Так у діалоговому вікні підтвердження заміни набору полиць:

Я вважаю, що правильна процедура - це Перша особа, яка вносить зміни та вимагає повторного перегляду. Коли ваш код потребує роботи, це означає, що ви будете його змінювати, тож вам захочеться мати стару версію для огляду для порівняння. Ви все ще маєте старий огляд в історії після його закриття, якщо хочете переглянути коментарі. Зараз ми перебуваємо в процесі оптимізації процесу перегляду коду на моєму робочому місці.

Я знаю, що це питання давнє, але воно все ще не підтримується, як вказували інші афіші. Рішення, запропоноване Чадом, буде працювати для деяких речей, але матиме дивну поведінку для інших.

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

Ви повинні зробити це за допомогою двох різних оглядів. Але є й спосіб зберегти історію за допомогою другого огляду. Все, що вам потрібно, - це завдання.

Цей робочий процес описаний для оглядів на основі змін, але він працює і для оглядів на полицях.

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

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