Haruskah kita menggunakan Tolak atau Tunggu Penulis di TFS ketika ada hal-hal yang harus diperbaiki?

12

Di TFS, ketika kami memberikan komentar untuk hal-hal yang harus diperbaiki dalam Permintaan Tarik sebelum kami menerimanya, haruskah kita menandainya sebagai Tolak atau Tunggu Penulis? Mana yang lebih baik?

masukkan deskripsi gambar di sini

sashoalm
sumber

Jawaban:

14

Sesuai Microsoft pada kode Tinjauan dengan permintaan tarik: Pilih perubahan tujuan yang disarankan dari setiap kelas persetujuan adalah:

Setujui dengan saran : Setuju dengan permintaan tarik, tetapi berikan saran opsional untuk meningkatkan kode.

Menunggu penulis : Jangan menyetujui perubahan, dan minta penulis untuk meninjau komentar Anda. Penulis harus memberi tahu Anda kapan Anda harus meninjau ulang kode setelah mereka mengatasi masalah Anda.

Ditolak : Perubahan tidak dapat diterima. Jika Anda memberikan suara dengan cara ini, Anda harus meninggalkan komentar di permintaan tarik yang menjelaskan mengapa perubahan itu ditolak.

Jadi saya Waiting for Authormaksudkan bahwa Anda pikir Penulis mengacaukan pendekatannya terhadap solusi tetapi kodenya dapat ditebus jika mereka menaruh komentar Anda dalam hati.

Dan Rejectedartinya Anda tidak akan menerima perubahan seperti ini di neraka, betapapun baiknya dituliskan kode tersebut.

Pertanyaan yang harus Anda tanyakan pada diri sendiri adalah apakah ide grup Anda tentang tujuan yang disarankan cocok dengan ide Microsoft.

Peter M
sumber
1
Tentang ini: "Penulis harus memberi tahu Anda ketika Anda harus meninjau ulang kode setelah mereka mengatasi masalah Anda." - adakah mekanisme bagi penulis untuk mengingatkan pengulas bahwa PR siap untuk ditinjau ulang?
Robert Sim
@RobertSim - yang kami gunakan adalah opsi 'Ping' di bawah Reviewer untuk mengirim pesan yang meminta ditinjau ulang.
killercowuk