Serbest bırakma ateşi sırasında etkili kod incelemesi nasıl yapılır?


17

Serbest bırakma için son tarih yarın, kolejiniz sonunda bu sürüm için çok önemli olan görevini bitirdi, proje yöneticisi omzunuzun üzerinde duruyor ve sonunda bir yapı yapmanız için baskı yapıyor ve inceleme sırasında kolej kodunuzda bir kusur fark edersiniz. Kritik değil, ama yarın serbest bırakılmasaydı gitmesine izin vermeyeceğiniz bir şey. Ve işleri daha da kötüleştirmek için, en kısa sürede bitirmeniz gereken kendi işiniz var. Ee ne yapıyorsun? Baskıya rağmen itirazınızı mı yükseltiyorsunuz yoksa sadece kaymasına izin veriyor musunuz?

Bulduğum bir yol, bu taahhüdü geçici olarak farklı bir dalda birleştirmek ve daha sonra incelemek. Sorun sadece kozmetik bir sorunsa ve hala kod incelemesi için bekleyen tek sorun ise işe yarar. Ancak bununla başa çıkmanın daha etkili bir yolu var mı? Örneğin, bir kişiyi yalnızca kod inceleme ve testlerine geçirmenizi tavsiye eder misiniz?


3
Neden sorunu gündeme getirmiyorsunuz ve yöneticinin bununla başa çıkmasına izin vermiyorsunuz? Sizinki değil, bu onun çağrısı gibi görünüyor: Ya potansiyel olarak hatalı bir uygulamayı serbest bırakmayı kabul ediyor ya da serbest bırakmayı (bir kısmını) geciktiriyor. Kaymasına izin vermek her iki dünyanın daha da kötüye gidiyor gibi görünüyor: bir hata sürümü yayınladınız ve bir kusur olduğunu bildiğinizden ve hiçbir şey söylemediğinden sorumlu oluyorsunuz.
Vincent Savard

1
Yönetici bu aramayı yapar, siz değil. Tabii, istisnayı yükselt. O zaman karar ver. Benim tahminim, bu sürümle devam edecek ve daha sonra ortaya koyduğunuz sorunla ilgilenmenize izin verecek.
Robert Harvey

Bir "akış" mı? Bir kusur mu demek istiyorsun?
Doc Brown

3
Ekibinizin haftada bir, ayda bir veya yılda bir yayın yapması büyük olasılıkla fark yaratacaktır.
Doc Brown

Serbest bırakılmadan kısa bir süre önce yapılan bu gözden geçirme toplantısında, insanlar normal koşullar altında bir şeylerin geçmesine izin vermek için çok daha hazırlıklı olacaklar - Ve gözden geçirildikten sonra, içeri giriyor. Bunu gerçekten istemiyorsunuz - Erteleyen tekliflere gidin serbest bırakıldıktan sonra herkes doğru aklına gelene kadar gözden geçirin.
Zaten ondan

Yanıtlar:


18

Bu sorunun cevabı iletişim kurmaktır.

  1. Sorun hakkında teknik / ekibe bilgi verin
  2. KG ile potansiyel etki hakkında konuşun
  3. Proje yönetimine (tam arkanızda olan), sürümün gecikmesine neden olan bir sorun olabileceğini söyleyin ve onlarla en kısa sürede geri döneceksiniz (birkaç dakika veya saat içinde)
  4. Bu sorunun sürüm için bir gösteri durdurucu olup olmadığını değerlendirin

Sorun bir gösteri durdurucu değilse, sürüme devam edin. KG'yi ve kullanıcılarınızı bu sorun hakkında bilgilendirin ve sorunun yamalanacağı bir takip tarihi belirleyin. Bu sadece üretime gitme riskini arttırır.

Bu takdirde ise bir gösteri tıpa (o alt satırında veya birinin sağlığı üzerinde olumsuz etkisi olacak olur) öneri salınımını geciktirmek ve neden anlatmak olduğunu zincirin yukarı bu iletişim kurar. Ardından geliştirici ile geri dönün ve sorunu düzeltmenin ne kadar süreceğini hesaplayın ve yönetime X dakika / saat / gün sayısına ihtiyacınız olduğunu söyleyin.

Şans yönetimi bu oyunda geç bir gösteri tıpa hakkında mutlu olmayacak, ama üretim için yayınlanmasını istemeyecektir.

İletişim kurun ve yönetimin çağrı yapmasına izin verin.


8

Sorunu sadece iletmekle kalmayın, belgeleyin

Şimdiye kadarki diğer cevaplarla ilgili büyük endişem: Bu satırlar boyunca, yakın bir tarihte karşılaşacak olan tipik proje yöneticisine söylediğiniz her şeyin göz ardı edilmesi veya unutulması muhtemeldir. Daha sonra, bir şeyler ters giderse, riski yetersiz bir şekilde iletmek için hala kancaya girebilirsiniz .

Proje yöneticisine bulduğunuz sorunu bildirin ve onu belgeleyeceğinizi bildirin . Durumunuzu göstermeniz gerekir.

Nerede belgelendirileceği ve kime söyleneceği çalışma ortamınıza bağlıdır, ancak kesinlikle patronunuzu ekleyin.

Risk ve Etkileri Belirleyin

Sorunun kritik bir sorun olmadığını söylüyorsunuz, ancak bunun ne anlama geldiğini gerçekten tanımlamıyorsunuz. Ondan kurtulmak bir sonraki adımınız.

Sorunu, bir soruna (riske) neden olma olasılığının ve riskin meyveye (etki) gelmesi durumunda sonuçların ciddiyetini tanımlayan hızlı bir risk ve etki analizi yapın . Yukarıdaki bağlantıda bulunan terimler gibi iyi tanımlanmış terimleri (proje yöneticinizin bilmesi gerekir) kullanın, ancak analizinizi yedekleyen bir açıklama da sağlayın.

Belgeleriniz ayrıca önerilen eylem yolunuzu da içermelidir . Evet , bir endişeyi dile getirmek ve yine de sürümle devam etmenizi öneririz. Riski tanımlamak doğrudur .

Bir sonraki sürümün ne zaman?

Risk / etki analizinizi tamamladıktan sonra ne önereceğiniz konusunda hala kararınız varsa, sürüm programınızı dikkate alın. Düzeltmeyi iki hafta içinde eklemeyi bekliyorsanız, bazı kusurlu kodlar serbest bırakılabilir.

Endişenizi çözmenin “deprioritize” olma olasılığı varsa (yani, bir sonraki parlak geliştirme lehine ihmal edilir), sorunu keşfettikten sonra sorunu mümkün olan en kısa sürede belgelemek için bir neden daha vardır: eğer etkili bir şekilde başlarsa saat ”konusuna bakın.


7

Bu durumda, herkesi bilgilendirin ve karar vermelerine izin verin. Muhtemelen serbest bıraktığınız ilk hata değil.

Son tarih yarın, ancak kendinizi bu durumda buluyorsunuz:

proje yöneticisi omzunun üzerinde duruyor ve sonunda bir yapı yapmaya zorluyor

Bu bir istisna olabilir, bu yüzden sadece bir hata geçtiği için işleminizde ciddi değişiklikler yapmayın. Yapmanız gereken bazı önlemler almaktır, bu yüzden son dakikada yapı yapmıyorsunuz. Umarım, başarılı olacağına dair güven vermek için yeterince düzenli bir yapı inşa edersiniz. Bu hala onları son dakikada çalıştırmak için yeterince iyi bir neden değil. İşlerin iyi test edilmesi, güveni artırmak için başka bir parçadır.

Bu senaryoyu bu şeye liderlik eden herkese sunun.

  • Ne tür problemlerin sunulması gerektiğini belirleyin.
  • Seçenekleri belirleyin.
  • Kararı kim veriyor
  • Tüm bunlara ne zaman karar verilmeli? Muhtemelen çıkış tarihine göreceli olacaktır.

Bu, bu son dakika sorunuyla ne yapacağınıza cevap vermese de, gelecekte onlarla başa çıkabileceğinizden emin olmanın bir yolunu sunar. Özellikle salıverilecek baskı olduğunda, şeyleri düşünülmüş ve duyguların yönlendirmediği bir şekilde ele almanın bir yoluna sahip olmak istersiniz.


1

Eğer bir son teslim tarihi varsa ve müdürünüz tam arkandaysa omzunun üstünden bakarsanız, ona gitmesini söylersiniz. O gidiyor ya da gidiyorsun. Ona baskı altında gözden geçirmeye zorlandığınızı, kodu gözden geçirmeyebileceğinizi açıklayın.

Böyle bir durumda, bazı kritik hataların geçeceğinden emin olabilirsiniz.

Yeni bir sürümü geri çekmek için birkaç gününüz olan Apple App Store'a göndermek gibi şanslı bir durumda olabilirsiniz. Ancak kodunuz müşterilere gönderilirse, bu bir felaket tarifi. Bir dahaki sefere yöneticinizin daha iyi planlar yapmasını umuyorum.


Bu cevabı küçümseyebileceğini anlayabiliyorum. Ancak bunun bir planlama sorunu olduğunu ve baskı altında gözden
geçirmenin

Bence OP'nin kelimenin tam anlamıyla "omzunun üstünden bakmak" anlamına gelmediği çok açık. IMHO bu cevap, sorunun anlamını tamamen gözden kaçırıyor.
Doc Brown

2
@DocBrown, bu cevabın noktayı kaçırdığına katılmıyorum. Ancak, PM kelimenin tam anlamıyla omzunun üzerinden bakarken, son gün orada gizlenen bir çok yer gerçekliktir.
Tim Grant

1

Diğer yanıtlar, kalite sürecinizi bükmeye çalışan yöneticinizin sorununa odaklanır.

Ancak, ne soruyorum düşünüyorum kod inceleme en az 2 kişi gerektirir ve bu nedenle önemli gecikmeler getiriyor aslında nasıl başa çıkmak olduğunu. Örneğin, bir görevin uygulanması 2 saat ve gözden geçirilmesi 2 saat alırsa, genellikle iki etkinlik arasında uzun bir duraklama olur. Gerçek zamanlı olarak görevin tamamlanması 2 veya 3 gün sürebilir.

Bu, klasik bir verim ve gecikme sorunudur. Yazılım üreten makinelerde (insanlar) bağlam anahtarları pahalıdır, bu nedenle birisinin işini derhal bir inceleme yapmasını önlemek yaygın bir uygulama olmamalıdır.

Sorunu azaltmak için bazı ipuçları:

  • Bir görev üzerinde çalışırken, inceleyen kişinin uzun süre zaman ayırması için kodun küçük parçalar halinde gönderilmesi gerekir.
  • Kod incelemeleri için yüksek öncelikli bir kültürü tanıtın. Bir istek aldığınızda mevcut iş biriminizi durdurmayın, ancak daha sonra ne yapacağınızı merak ediyorsanız, her zaman sıranızdan bir inceleme seçin.
  • Varsa, takımlarınızdaki saat dilimi farkından yararlanın.
  • Tek bir kişiyi yalnızca kod incelemesine adamayın. Çelişkili. Görevi konusunda ciddi olacaksa yükü kaldıramayacak. Yöneticiniz sadece bir gün tasarruf etmek için birinin boşta olduğu fikrini kabul etmeyecektir.
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.