[Gelistirici] Review sürecindeki ACK verme üslubu

Fatih Aşıcı fatih at pardus.org.tr
8 Ara 2010 Çar 09:18:30 EET


On Wednesday 08 December 2010 08:50:01 Fatih Arslan wrote:
> Merhaba,
> 
> Review mesajlarında sıkça karşılaştığım bir davranış var. Paket
> sıkca bakıcıları şu kalıbı kullandığını gördüm: "Şurada şu eksiği
> değiştirdikten sonra ACK". Review sürecini anlatan belgemizde bu
> davranıştan kaçınılması gerektiği yazılıyor [1]
> 
> ...
> 
>    Örneğin: YANLIŞ: Dosya izinlerini değiştirdikten sonra ACK.
>             DOĞRU : Dosya izinlerinin düzenlenmesi gerek.
> 
> ...
> 
> Review süreciyle ilgili bir değişiklik mi var ? Yoksa çok küçük şeyler
> için, örneğin COPYING dosyası eklenmemesi durumda _YANLIŞ_ olan
> yöntem'e göre mi hareket edilmeli ? Yeni kişilerin de bu davranışı
> benimsediğini ve kullandığını gördüm (buna ben de dahilim).
> 
> Fakat süreci belirtilmiş bir prosedüre aykırı davranmak benim hoşuma
> gitmiyor. Çünkü belirsizlik ortaya çıkıyor ve bu yüzden review
> ederken "acaba şöyle mi yapsaydım..." düşünceleri içinde boğulup
> duruyorum.
> 
> Bu konuda nasıl hareket edilmeli ?

Küçük şeylerin sınırını çizmek çok zor. O yüzden belgede anlatıldığı gibi 
yapalım.
-------------- sonraki bölüm --------------
A non-text attachment was scrubbed...
Name: kullanılamıyor
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://liste.pardus.org.tr/gelistirici/attachments/20101208/7ec02fec/attachment-0002.pgp>


Gelistirici mesaj listesiyle ilgili daha fazla bilgi