TFS Kod İncelemeleri - Yorumlara yanıt olarak güncellenmiş dosyaları göster


127

VS 2012 ve VS 2013 önizlemesinde yerleşik olarak bulunan kod inceleme işlevini kullanmaya başlıyoruz. İnceleme istemek ve yorum eklemek oldukça basit görünüyor. Birisi kodun değiştirilmesini isteyen yorumlar eklerse, talepte bulunan kişi bu değişiklikleri nasıl yapar ve gösterir?

Dolayısıyla süreç şu şekilde akacaktır:

  1. Kişi 1, bir kod incelemesi ister.
  2. 2. Kişi yorumlar ekler ve "Çalışılması Gerekiyor" u seçer.
  3. Kişi 1 gerekli değişiklikleri yapar.

Kişi 1 şimdi bu değişiklikleri 2. Kişiye nasıl gösteriyor? Yorum ekleyebilir ve gönderebilirsiniz, ancak dosyalar değişmez. Dosyaların, orijinal inceleme istendiğinde oluşturulan değişiklik kümesinden olduğunu varsayıyorum. Kişi 1 bu incelemeyi kapatmalı ve ikinci bir inceleme talep etmeli mi?

İkinci bir gözden geçirme oluşturmak pek uygun görünmüyor, çünkü değişiklikleri neden yaptığınıza dair konuşmanızın geçmişini kaybediyorsunuz.

Kod incelemelerinin temel işlevlerinin nasıl kullanılacağını gösteren tonlarca web sitesi var, ancak en iyi uygulamaları gösteren herhangi bir site var mı? Örneğin, dosyaların yanındaki onay kutularına kim tıklamalı?


"En iyi uygulamaları gösteren siteler var mı" için +1. ALM Rangers, genellikle derinlemesine kullanım için "gitme" adamımdır, ancak bulabildiğim tek bilgi, sorunuzu yanıtlamaz. Kod İncelemesini kullanmak için bazı öneriler ve senaryolar görmeyi çok isterim.
DaveShaw

Yanıtlar:


164

Dolayısıyla süreç şu şekilde akacaktır:

  1. Kişi 1, bir kod incelemesi ister.
  2. 2. Kişi yorumlar ekler ve "Çalışılması Gerekiyor" u seçer.
  3. Kişi 1 gerekli değişiklikleri yapar.
  4. Kişi 1 Kod incelemesiyle ilişkili raf kümesini günceller
  5. 1. Kişi tartışmaya devam etmek için yorumlar ekler
  6. Kabul edilene kadar 2-5. Adımları tekrarlayın

İncelemeyle ilişkili raf setini güncellemek için gerekli adımlar aşağıda verilmiştir.

  1. "Kod İnceleme" bölmesinden "raf kümesini görüntüle" bağlantısını seçin
  2. "Raf Seti Ayrıntıları" bölmesinden raf seti adını vurgulayın ve kopyalayın
  3. "Bekleyen Değişiklikler" bölmesine gidin, "Raf" ı tıklayın ve raf seti adını yapıştırın
  4. Raf seti değiştirme doğrulama iletişim kutusundaki Evet düğmesine basın
  5. Şimdi gözden geçiren güncellenen dosyaları görebilir ve inceleme tartışması devam edebilir

Bazı ekran görüntülerini ekledim, çünkü bazı şeyleri netleştirmeye yardımcı oluyor.


1) "Kod İnceleme" bölmesinden, burada gösterildiği gibi "raf kümesini görüntüle" bağlantısını seçin:

görüntü açıklamasını buraya girin


2) "Raf Seti Ayrıntıları" bölmesinden, burada gösterildiği gibi raf seti adını vurgulayın ve kopyalayın:

görüntü açıklamasını buraya girin


3) "Bekleyen Değişiklikler" bölmesine gidin, "Raf" ı tıklayın ve raf seti adını yapıştırın, örneğin:

görüntü açıklamasını buraya girin


4) Raf seti değiştirme doğrulama iletişim kutusundaki Evet düğmesine basın:

görüntü açıklamasını buraya girin


7
Bu çözümle ilgili sorun, tüm yorumların uçup gitmesi açısından yıkıcı olmasıdır. Bence daha iyi bir çözüm, yeni bir kod incelemesi oluşturmak ve eskisini ilgili bir öğe olarak ayarlamak olacaktır.
Bardia

15
Raf setini değiştirmek yorumlara yalnızca dosyalara dokunmaz. Ekibimiz bu yöntemi başarıyla kullanıyor ve herhangi bir inceleme yorumunu kaybetmiyoruz.
Chad Dienhart

10
Bu işe yarar, ancak bir dosyayı düzenleyip raf kümesini değiştirdiğinizde satır ekler / kaldırırsanız, o dosyadaki yorumlar yanlış satırlara işaret eder.
Chin

2
Genellikle tekrar etmekten hoşlanmam, ancak @ Chin'in yukarıdaki noktası bu yanıtı tamamen kabul edilemez kılıyor. Evet, raf kümenizi kolayca güncelleyebilirsiniz, ancak bunu yapmak esasen TÜM önceki inceleme yorumlarının artık YANLIŞ metne işaret etmesine neden olur. (Görünüşe göre bir inceleme yorumu bir dosyanın başlangıcından itibaren mutlak bir denkleştirmeye eklenmiş.)
Michael Sorens

7
Fazla manuel ve ideal bir şey değil. TFS veya Visual Studio, ardışık incelemeler arasındaki değişiklikleri karşılaştıracak bir şeye sahip olmalıdır, böylece gözden geçiren son incelemeden bu yana yapılan değişiklikleri tam olarak görebilir. Böyle bir şey yok mu? Belki bir eklenti veya başka bir şey ...
Iúri dos Anjos

6

Doğru prosedürün, 1. Kişinin değişiklikleri yapması ve başka bir inceleme talep etmesi gerektiğine inanıyorum. Kodunuzun çalışması gerektiğinde, bu, onu değiştireceğiniz anlamına gelir, bu nedenle karşılaştırma için eski sürüme bakmak isteyeceksiniz. Yorumlara göz atmak isterseniz, kapatıldıktan sonra da geçmişte eski incelemeye sahipsiniz. Şu anda iş yerimde kod inceleme sürecimizi optimize etme sürecindeyiz.


2

Bu sorunun eski olduğunu biliyorum, ancak yine de diğer afişlerde belirtildiği gibi desteklenmiyor. Chad tarafından önerilen çözüm bazı şeyler için işe yarayacak, ancak diğerleri için tuhaf davranışlar sergileyecektir.

Kısa bir süre önce, TFS ekibi, Kod İncelemelerinin yinelemelere sahip olmasını sağlayarak bu sorunu zarif bir şekilde çözmesi gereken bir Kod İncelemesinin güncellenmesini sağlamak için çok eski bir kullanım faturası talebine yönelik bir çözüm için planlama aşamalarına başladı .


1
Harika. Ben de dört gözle bekleyeceğim. Gerçek bir çözümü memnuniyetle karşılıyorum. Yazım, problemi çözmenin bir yolu.
Chad Dienhart

Umarım gök mavisi DevOps çekme istekleri gibi görünmeyecektir. Bunlar spam dolu bir kabustur.
StingyJack

0

Bunu iki farklı inceleme ile yapmanız gerekiyor. Ancak ikinci gözden geçirme ile tarihi tutmanın bir yolu da var. Tek ihtiyacınız olan görevler.

Bu iş akışı, değişiklik kümesi tabanlı incelemeler için açıklanmıştır, ancak raf tabanlı incelemeler için de çalışır.

  1. Görev 1 oluştur
  2. Değişiklikleri kontrol etmeden önce set1, task1'i ilgili bir çalışma öğesi olarak ekleyin
  3. Bir iş öğesi ile ilgili değişiklikleri kontrol edin ve bu değişiklik kümesinin gözden geçirilmesini isteyin
  4. Görev2 oluştur
  5. Değişiklikleri kontrol etmeden önceet2 her iki görevi de ilgili çalışma öğesi olarak ekleyin.
  6. İlgili iki iş öğesi ile değişiklikleri kontrol edin ve bu değişiklik kümesinin gözden geçirilmesini isteyin

Şimdi ikinci gözden geçirme talebinde gözden geçiren ilgili görevleri arayabilir ve gözden geçiren görev1'i ararsa değişiklikleriet1 ve yorumlarıyla birlikte gözden geçirme isteğini görür. Böylece konuşma geçmişini kaybetmeyeceksiniz.

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.