Ulasan Kode TFS - Tampilkan file yang diperbarui dalam menanggapi komentar


127

Kami mulai menggunakan fungsionalitas peninjauan kode bawaan untuk VS 2012 dan VS 2013 pratinjau. Meminta peninjauan dan menambahkan komentar tampaknya cukup mudah. Jika seseorang menambahkan komentar yang meminta kode untuk diubah, lalu bagaimana pemohon membuat perubahan ini dan menunjukkannya?

Jadi prosesnya akan mengalir seperti ini:

  1. Orang 1 meminta peninjauan kode.
  2. Orang 2 menambahkan komentar dan memilih "Butuh Pekerjaan."
  3. Orang 1 membuat perubahan yang diperlukan.

Bagaimana Orang 1 sekarang menunjukkan perubahan ini ke Orang 2? Anda dapat menambahkan komentar dan mengirimkannya, tetapi file tidak berubah. Saya berasumsi file-file tersebut berasal dari changeset yang dibuat ketika ulasan asli diminta. Haruskah Orang 1 menutup ulasan ini, dan meminta tinjauan kedua?

Membuat ulasan kedua sepertinya tidak optimal, karena Anda kehilangan riwayat percakapan Anda tentang mengapa Anda melakukan perubahan.

Ada banyak situs web yang menunjukkan cara menggunakan fungsi dasar ulasan kode, tetapi apakah ada situs yang menunjukkan praktik terbaik? Misalnya, siapa yang harus mengklik kotak centang di sebelah file?


+1 untuk "apakah ada situs yang menunjukkan praktik terbaik". Rangers ALM biasanya adalah "goto guys" saya untuk penggunaan mendalam, tetapi satu-satunya informasi yang saya dapat temukan, tidak menjawab pertanyaan Anda. Saya akan senang melihat beberapa pemikiran dan skenario untuk menggunakan Peninjauan Kode.
DaveShaw

Jawaban:


164

Jadi prosesnya akan mengalir seperti ini:

  1. Orang 1 meminta peninjauan kode.
  2. Orang 2 menambahkan komentar dan memilih "Butuh Pekerjaan."
  3. Orang 1 membuat perubahan yang diperlukan.
  4. Orang 1 Memperbarui rak yang terkait dengan tinjauan kode
  5. Orang 1 menambahkan komentar untuk melanjutkan diskusi
  6. Ulangi langkah 2 - 5 hingga diterima

Berikut adalah langkah-langkah yang diperlukan untuk memperbarui rak yang terkait dengan ulasan.

  1. Dari panel "Tinjauan Kode" pilih tautan "view shelveset"
  2. Dari panel "Detail Shelveset" sorot dan salin nama shelveset
  3. Arahkan ke panel "Pending Changes", klik "Shelve" dan rekatkan nama shelveset
  4. Tekan tombol Ya di dialog shelveset replace verifikasi
  5. Sekarang reviewer dapat melihat file yang diperbarui dan diskusi review dapat dilanjutkan

Saya telah memasukkan beberapa tangkapan layar karena saya merasa ada gunanya untuk mengklarifikasi hal-hal.


1) Dari panel "Tinjauan Kode" pilih tautan "view shelveset" seperti yang ditunjukkan di sini:

masukkan deskripsi gambar di sini


2) Dari panel "Detail Shelveset" sorot dan salin nama shelveset seperti yang ditunjukkan di sini:

masukkan deskripsi gambar di sini


3) Navigasikan ke panel "Pending Changes", klik "Shelve" dan tempel nama shelveset misalnya:

masukkan deskripsi gambar di sini


4) Tekan tombol Ya di dialog shelveset replace verifikasi:

masukkan deskripsi gambar di sini


7
Masalah dengan solusi ini adalah bahwa itu merusak karena semua komentar terpesona. Saya pikir solusi yang lebih baik adalah membuat review kode baru dan menetapkan yang lama sebagai item terkait.
Bardia

15
Mengganti rak tidak hanya menyentuh komentar file. Tim kami berhasil menggunakan metode ini dan kami tidak kehilangan komentar ulasan.
Chad Dienhart

10
Ini berfungsi, tetapi jika Anda menambahkan / menghapus baris ketika Anda mengedit file dan mengganti shelveset, komentar yang ada di file itu akan mengarah ke baris yang salah.
Dagu

2
Saya biasanya tidak suka mengulang, tapi poin @ Chin di atas membuat jawaban ini benar-benar tidak bisa dipertahankan. Ya, Anda dapat dengan mudah memperbarui set rak Anda, tetapi melakukan itu pada dasarnya SEMUA komentar ulasan sebelumnya sekarang menunjuk ke teks SALAH. (Karena ternyata komentar ulasan terlampir pada offset absolut dari awal file.)
Michael Sorens

7
Terlalu manual dan bukan hal yang ideal. TFS atau Visual Studio harus memiliki sesuatu untuk membandingkan perubahan antara ulasan berurutan, sehingga peninjau dapat melihat perubahan yang dibuat sejak ulasan terakhir. Tidak ada yang seperti ini? Mungkin sebuah plugin atau sesuatu ...
Iúri dos Anjos

6

Saya percaya prosedur yang benar adalah bagi Orang 1 untuk membuat perubahan dan meminta peninjauan kembali. Ketika kode Anda membutuhkan pekerjaan, itu berarti Anda akan mengubahnya sehingga Anda ingin memiliki versi lama untuk melihat kembali untuk perbandingan. Anda masih memiliki ulasan lama dalam riwayat setelah ditutup jika Anda ingin melihat komentar. Kami saat ini sedang dalam proses mengoptimalkan proses peninjauan kode kami di tempat kerja saya.


2

Saya tahu pertanyaan ini sudah lama, tetapi masih belum didukung sebagaimana ditunjukkan oleh poster lain. Solusi yang diusulkan oleh chad akan bekerja untuk beberapa hal tetapi akan memiliki perilaku aneh untuk orang lain.

Baru-baru ini, tim TFS memulai tahap perencanaan untuk solusi permintaan uservoice yang sangat lama untuk memungkinkan pembaruan Tinjauan Kode yang harus menyelesaikan masalah ini secara elegan dengan membuat Tinjauan Kode memiliki iterasi.


1
Bagus. Saya juga akan menantikannya. Saya akan menyambut solusi nyata. Posting saya hanyalah cara untuk mengatasi masalah tersebut.
Chad Dienhart

Mudah-mudahan itu tidak akan terlihat seperti permintaan tarik biru DevOps. Mereka adalah mimpi buruk spam.
StingyJack

0

Anda harus melakukan ini dengan dua ulasan berbeda. Tetapi ada juga cara untuk menjaga sejarah dengan ulasan kedua. Yang Anda butuhkan hanyalah tugas.

Alur kerja ini dijelaskan untuk ulasan berbasis perubahan, tetapi juga berfungsi untuk ulasan berbasis rak.

  1. Buat tugas1
  2. Sebelum memeriksa perubahan, tambahkan tugas1 sebagai item pekerjaan terkait
  3. Periksa perubahan dengan satu item yang terkait dengan pekerjaan dan minta ditinjau untuk perubahan ini
  4. Buat tugas2
  5. Sebelum memeriksa perubahan, tambahkan kedua tugas sebagai item pekerjaan terkait.
  6. Periksa perubahan dengan dua item pekerjaan terkait dan minta ditinjau untuk perubahan ini

Sekarang dalam permintaan review kedua reviewer dapat mencari tugas terkait dan jika reviewer mencari task1 dia melihat perubahan1 dan permintaan review dengan komentar-komentarnya. Jadi, Anda tidak akan kehilangan riwayat percakapan.

Dengan menggunakan situs kami, Anda mengakui telah membaca dan memahami Kebijakan Cookie dan Kebijakan Privasi kami.
Licensed under cc by-sa 3.0 with attribution required.