Çekme isteği incelemesi alma

Çekme isteğiniz gönderildi ve CI'yı geçti. Artık incelenmeye hazır.

tl;dr - İnceleme süreci

İnceleme sürecinin kısa versiyonu:

1. İncelemeyi bekleyin.
2. Geri bildirimlere yanıt verin.
3. Değişiklik talep edilmesi durumunda:
4. İstenen değişiklikleri tamamlayarak çalışın.
5. İstenen tüm değişiklikleri gönderin.
6. İstenen tüm değişiklikler gönderildikten sonra yeniden inceleme talebinde bulunun.
7. Değişiklik gerekmeyene kadar üçüncü bölümü tekrarlayın.
8. Pull isteğinizin onaylanıp birleştirilmesini bekleyin.

Tebrikler! BeeWare'ye katkıda bulundunuz!

Çekme isteğimi gönderdim, şimdi ne olacak?

Çekme isteğinizi gönderdikten sonra, katkınızın incelenmesini beklemeniz gerekir. İnceleme sürecinin iki yönü vardır: inceleme sağlamak ve inceleme almak.

Beklentileri gözden geçirin

Gönderilerinizi inceleyen herkesin, çekirdek ekip üyelerinin incelemeleri de dahil olmak üzere, bu yönergeleri izleyeceğini beklemelisiniz. Başkalarının gönderilerini incelerken de bu yönergeleri izlemelisiniz.

İnceleme yapan kişinin bu beklentilerden saptığını düşünüyorsanız ve pull isteğinde bu konuyu kendiniz gündeme getirmekten çekinmiyorsanız, bunu yapabilirsiniz. Eğer kendinizi rahat hissetmiyorsanız, lütfen BeeWare Davranış Kuralları Müdahale Ekibi ile iletişime geçin. Bildiriminizi inceleyeceğiz ve gözden geçiren kişiyle görüşeceğiz. Görüşme, bildirilen eylemi yansıtacaktır; küçük bir ihlal tartışma ile sonuçlanabilirken, büyük bir ihlal daha ciddi sonuçlara yol açabilir.

İnceleme sağlamak

Herkes herhangi bir çekme isteği hakkında inceleme yapabilir. Bu kılavuzlar, çekirdek ekip üyesi veya topluluk üyesi tarafından yapılıp yapılmadığına bakılmaksızın, incelemeden beklentilerimizi açıklamaktadır.

Çekirdek ekip üyesi her zaman son incelemeyi yapmak zorundadır; ancak topluluk üyelerinin incelemeleri süreci kolaylaştırmak için yararlı olabilir - ideal olarak, topluluk incelemeleri tüm önemli sorunları belirledikten sonra çekirdek ekibin incelemesi bir formalite haline gelir.

İnceleme alma

Bir inceleme almak üç temel adımı içerir:

1. İlk geri bildirimler ve sorular.
2. Değişiklik talepleri.
3. Onay ve birleştirme.

Her adım aşağıda ayrıntılı olarak açıklanmıştır. Süreç boyunca herhangi bir noktada sorunuz olursa, çekinmeden sorun! Size yardımcı olmaktan memnuniyet duyarız.

Zaman çizelgesi ve ilk geri bildirimler

Çekirdek ekip, her çekme isteğinin on iş günü içinde incelenmesini sağlamayı amaçlamaktadır. Ancak, daha karmaşık gönderimler söz konusu olduğunda veya çekme isteği ekibin bir kısmı izinliyken gönderildiğinde, bu süre uzayabilir.

Genellikle her çekme isteğinde gözden geçirenlerle sürekliliği koruruz, yani tüm inceleme süreci boyunca muhtemelen aynı gözden geçirenle çalışacaksınız. Bu, gözden geçireninizin süreç boyunca bağlamı anlayacağı ve yanıt sıklığı ve inceleme stili açısından ne beklemeniz gerektiğini öğrenebileceğiniz anlamına gelir. İlk gözden geçiren, pull isteklerinizi gözden geçirmek için gerekli uzmanlığa sahip olmadığını belirler veya herhangi bir nedenle bu görevi yerine getiremeyeceğini anlarsa, pull isteğinizle ilgili sorumluluğu başka bir ekip üyesine devredebilir.

Her bir yazışmaya on iş günü içinde yanıt vermemizi bekleyebilirsiniz. Geri bildirimlere ve sorulara yanıt vermek, inceleme sürecinin önemli bir parçasıdır. Sürecin bir sonraki adımına geçmeden önce sizden bir yanıt bekliyoruz.

Değişiklik talepleri

Çoğu zaman, inceleme görevlisi pull isteğinizde değişiklik talep edecektir. Bu, mutlaka çalışmanızın bir yansıması değildir, sadece sürecin bir parçasıdır.

İlk inceleme önemli sayıda sorun ortaya çıkarsa, ilk inceleme kapsamlı olmayabilir. Bunun yerine, pull isteğini birleştirilebilir bir duruma getirmek için gereken çalışma hakkında üst düzey yönlendirme sağlamaya odaklanacaktır. İnceleme süreci, denenen çalışmanın amacını ve kapsamını netleştirmek için sorular içerebilir.

İstenen değişiklikleri yapın

İnceleme görevlisi, çekme isteğinize yorumlar ekleyecektir. Bu yorumlar genel olabilir, belirli bir dosya veya belirli bir satır ya da satırlarla ilgili olabilir. Bazen, GitHub kullanıcı arayüzü aracılığıyla çekme isteğinize uygulayabileceğiniz doğrudan önerilen değişiklikleri de içerebilir. Genellikle, sorular, açıklama talepleri veya güncellemelerle ilgili kılavuzlar olacaktır.

Bir konuşmayı çözülmüş olarak işaretleme

Geri bildirim sürecinin tartışma bölümünde, değerlendiriciniz tarafından başlatılan bir konuşmayı asla "çözüldü" olarak işaretlememelisiniz. Konuşmayı çözüldü olarak işaretlemek değerlendiricinin sorumluluğundadır. Belirlenen sorunun çözülüp çözülmediğini belirlemek onlara kalmıştır.

İnceleme sistematik bir sorun ortaya çıkarsa (örneğin, kodda isim tutarsızlığı varsa), incelemeyi yapan kişi bu sorunun her örneğini vurgulamayabilir. Bunun yerine, sorunun birkaç örneğini seçip diğer örneklerin de düzeltilmesi gerektiğini belirtebilir. İnceleme bir yerde bir sorunu vurgularsa ve siz bunun başka yerlerde de geçerli olabileceğini düşünüyorsanız, bu sorunu nerede olursa olsun düzeltmelisiniz. Emin değilseniz, incelemeyi yapan kişiden açıklama isteyin.

İstenen tüm değişiklikleri gönderin

İstenen tüm değişiklikleri yaptıktan sonra, pull isteğinize bir güncelleme gönderebilirsiniz. Bu, yeni bir CI çalıştırmasını tetikleyecektir; CI'nın hala başarılı olduğunu onayladıktan sonra, güncellenmiş bir inceleme talep eden bir yorum gönderin ve çekirdek ekip pull isteğinizi tekrar inceleyecektir.

İt, zorlama veya yeniden temel alma

İnceleme sırasında çekme isteğinizi güncellerken, taahhüt geçmişini olduğu gibi bırakmak önemlidir. Taahhütlerin sayısı çok fazla olsa da sorun değildir; çekme isteğini birleştirdiğimizde hepsi birleştirilir. İnceleme sırasında çekme isteğinizi zorla gönderir veya yeniden temel alırsanız, incelemeyi yapan kişinin ihtiyaç duyduğu önemli bağlamı ortadan kaldırmış olabilirsiniz.

Yeniden inceleme talebinde bulun

Belirli bir incelemede istenen tüm değişiklikleri çözdükten ve CI tekrar geçtiğinde, inceleyicinizden yeniden inceleme talep edebilirsiniz. Bir sorun özellikle karmaşıksa ve bir sorunu düzeltmek başka bir şeyi etkileyecekse, güncellediğiniz belirli parça için inceleme talep edebilirsiniz. Her inceleme talebinin tam bir inceleme talebi olduğu varsayılacaktır. Tam incelemeye hazır değilseniz, tam olarak ne istediğinizi belirtmeyi unutmayın.

Çekme isteği onayı ve birleştirme

Tüm değişiklik taleplerine yanıt verdikten sonra, çekme talebi onaylanacaktır. Çoğu durumda, bir çekme talebi onaylandığında, hemen birleştiririz. Bazı durumlarda, henüz birleştirilmemiş başka bir çekme talebine bağlı olmak gibi, gecikmeye neden olacak hafifletici koşullar olabilir. Bu durumu yorumlarda bildiririz, böylece durumu öğrenebilirsiniz.