Düzeltilecek şeyler olduğunda TFS'de Reddet veya Yazar Bekle'yi kullanmalı mıyız?


13

TFS'de, kabul etmeden önce bir Çekme İsteğinde düzeltilecek şeyler için yorum koyduğumuzda, Reddet veya Yazar Bekle olarak işaretlemeli miyiz? Hangisi daha iyi?

resim açıklamasını buraya girin

Yanıtlar:


15

Çekme istekleriyle İnceleme kodunda Microsoft'a göre : Değişikliklere oy verin, her onay sınıfının önerilen amacı:

Önerilerle onaylayın : Çekme talebini kabul edin, ancak kodu geliştirmek için isteğe bağlı öneriler sağlayın.

Yazarı beklemek : Değişiklikleri onaylamayın ve yazardan yorumlarınızı incelemesini isteyin. Yazar, endişelerinizi giderdikten sonra kodu ne zaman yeniden gözden geçirmeniz gerektiğini size bildirmelidir.

Reddedildi : Değişiklikler kabul edilemez. Bu şekilde oy veriyorsanız, çekme isteğinde değişikliklerin neden reddedildiğini açıklayan bir yorum bırakmalısınız.

Bu nedenle Waiting for Author, yazarın çözüme yaklaşımında berbat olduğunu düşündüğünüzü, ancak yorumlarınızı kalbe götürürse kodunun kullanılabilir olduğunu kastediyorum.

Ve Rejectedkodun ne kadar iyi yazıldığına bakılmaksızın böyle bir değişikliği kabul etmediğiniz anlamına gelir.

Kendinize sormanız gereken soru, gruplarınızın önerilen amaç fikrinin Microsoft'un fikrine uyup uymadığıdır.


1
Bu konuda: "Yazar, endişelerinizi giderdikten sonra kodu ne zaman yeniden gözden geçirmeniz gerektiğini size bildirmelidir." - yazarın PR'ları yeniden gözden geçirmeye hazır olduğu konusunda yorumcuları uyarması için bir mekanizma var mı?
Robert Sim

@RobertSim - yeniden gözden geçirme isteyen bir mesaj göndermek için Reviewers altındaki 'Ping' seçeneğini kullanıyoruz.
killercowuk

Buna bakmanın başka bir yolu: Waiting for Author= Bu özelliği istiyorum, ancak yanlış kodladınız ve önerdiğim şekilde düzeltene kadar onaylamayacağım. Ve Rejected= İncelememi istediğin bu özelliği istemiyorum.
goku_da_master
Sitemizi kullandığınızda şunları okuyup anladığınızı kabul etmiş olursunuz: Çerez Politikası ve Gizlilik Politikası.
Licensed under cc by-sa 3.0 with attribution required.