Goller
İnceleme sürecimizin birkaç hedefi vardır:
- Kodun hem işlevsellik hem de okunabilirlik açısından yüksek kaliteli olduğundan emin olun.
- Hataları yakalayın, çünkü hatalar olabilir.
- Kod tabanının herhangi bir bölümünde çalışmaya başlamayı kolaylaştırmak için tutarlı bir stili koruyun.
blockly-samples ve core Blockly içerisine giren tüm kodlar, ister topluluğa katkıda bulunan kişiler ister Blockly ekip üyeleri tarafından yazılmış olsun), incelemeden geçer.
İncelemeciler olarak, değişikliklerinizi mümkün olduğunca iyi hale getirmek için sizinle birlikte çalışmayı hedefliyoruz. Katkıda bulunanlar olarak sizden, alma taleplerinizin inceleme ve birleştirme yoluyla iletilmesi için bizimle iletişime geçmenizi rica ediyoruz.
İşlem
Halkla ilişkiler inceleme süreci birkaç aşamadan oluşur:
Ödev
Pull isteğiniz geldiğinde Blockly ekibinin Nöbetçi üyesi bir incelemeci atar.
İncelemeciler uzmanlığına göre ve iş yükünü eşit şekilde dağıtmak için seçilir.
İnceleme uzmanının atanması birkaç gün, incelemenin alınması ise birkaç gün daha sürebilir. Endişelenmeyin, bu durum normaldir.
Geri bildirim
Geri bildirim aşamasında bir incelemeci PR'nizde değişiklikler için öneriler bırakır. Bunlar, kodunuzu Google JavaScript stil kılavuzuna uygun hale getirmek için basit işlemler olabilir. Veya fonksiyon tanımlarınızı yeniden düzenlemenizi istemek gibi daha büyük görevler de olabilir.
İncelemecilerin, birkaç bildirim yerine tek bir bildirim almak için (ayrı ayrı yorumlar yapmak yerine) GitHub kod incelemelerini kullanmaları önerilir.
Tartışma
Tartışma aşamasında geri bildirime yanıt verebilirsiniz. Belki yorum yorumlarından biri yeterince açık değildir: Şimdi size açıklama isteyebilirsiniz. Ya da belki inceleyen kişi değişiklik talebinde bulunmuş, ama siz bunun sonuçları olacağını düşünüyorsunuzdur. Artık uzlaşma için bir fırsat var.
Düzeltme
Revizyon aşamasında, halkla ilişkilernizde değişiklik yapabilirsiniz. Bu değişiklikler genellikle incelemecinizin geri bildirim aşamasında söylediği bir şeyin sonucudur.
Düzeltmelerinizi tamamladıktan sonra, inceleyen kişiyi etiketleyerek söz konusu kişiye tekrar göz atmalarını önerebilirsiniz.
Tekrar
Düzeltme aşamasından sonra incelemecinin geri bildirim vermek için bir şansı daha olur ve süreç baştan başlar.
Genellikle ikinci inceleme basittir ve noktalama işaretleri ve kod stili gibi nitlere odaklanır. Ancak bazen ikinci bir inceleme oldukça büyük olabilir. İlk yorumcunuz, yeni bir bakış açısından bakmak bile isteyebilir.
Birleştir!
Birleştirme aşamasında da kutlama fırsatı elde edersiniz. Bir değişiklik oluşturdunuz, bu değişiklik üzerinde durdunuz, düzelttiniz ve nihayet bu değişikliği birleştirdiniz! Bu, pek çok kişinin tamamlaması bir yana, Asla başlatmadığı büyük bir başarı.
Blockly'yi daha iyi hale getirmek üzere gösterdiğiniz yoğun çaba için teşekkür ederiz. Ayrıca tebrikler!