Pull istekleri, bir deponun can kanı gibidir. Her şeyin sağlıklı ve hareketli kalmasını sağlarlar. Bu sayfada, PR'nizin birleştirilme olasılığını artırmak için eksiksiz ve incelenmesi kolay bir PR'nin nasıl oluşturulacağı ayrıntılı bir şekilde açıklanmaktadır.
Mümkün olan en iyi PR'yi oluşturmak için uygulayabileceğiniz adımlar şunlardır:
Bilgi verin.
Başlamadan ve kod yazmaya başlamadan önce çekirdek ekiple iletişime geçmeniz faydalı olacaktır. Böylece, ilgi alanlarınızı öğrenebilirler.
İlginizi çeken bir sorun varsa, o sorun üzerinde çalışmaya başlayacağınızı söyleyen bir yorum ekleyin. Böylece aynı şey üzerinde çalışan birden fazla kişi olmaz. Bir ekip üyesi, o kişinin size ait olduğunu onaylamak için cevap verir.
Sorun kapsamında olmayan bir fikriniz varsa çalışmaya başlamadan önce lütfen bir fikriniz yazın. Böylece ekip, geliştirmeye başlamadan önce değişikliği en iyi nasıl oluşturabileceklerini tartışma fırsatı bulur. Bu da uzun vadede iş gücünden tasarruf etmenizi sağlar.
Hazırlanın
Bloklu veya bloklu örneklere ilk kez katkıda bulunuyorsanız geliştirme kurulumu sayfasından başlayın.
Küçük tutun
Yaptığınız değişikliklerin her zaman küçük ve odaklı olmasına çalışın. Tek bir büyük halkla ilişkiler (PR) incelemesinden ziyade, birden fazla küçük PR'yi incelemeyi tercih ederiz. Temel kurallardan bazıları şunlardır:
- Tek bir sorunu çözmek. Birden çok sorunu aynı anda ele almaya çalışmayın.
- Kapsamı sınırlama. Halkla ilişkiler (PR) genellikle 8 saatten kısa sürer (kod tabanını ne kadar iyi olduğunuza bağlı olarak).
- Kayıtları kullanın. PR'niz biraz büyük görünüyorsa değişiklikleri git kaydetme özelliğini kullanarak mantıksal gruplara ayırın.
Temiz Tutun
Kod stili neden önemlidir? Uzun vadede bu işimiz var ve tutarlı bir tarz, bakımı kolaylaştırır. Stil, değişkenlerinizi nasıl adlandırdığınız ile ilgilidir, aynı zamanda kodunuzu nasıl yapılandırdığınızı, yorum yazdığınızı ve daha fazlasını da kapsar. Mümkün olduğunda, stil kontrollerini otomatikleştirmek için eslint gibi araçlar kullanırız.
Eslint'e ek olarak lütfen şu kılavuzları da uygulayın:
- Google JavaScript stil kılavuzu.
- Mesaj kaydetme kılavuzu
- Blockly API'leri kullanma kuralları
- Codelab stil kılavuzu
Değişikliğinizi Test Edin
PR hazırlamadan önce değişikliklerinizin işe yarayıp yaramadığını her zaman test etmelisiniz. Böylece, geri dönüp düzeltme yapmak zorunda kalmazsınız. Farklı proje kategorilerini test etmek için bazı fikirler şöyle olabilir:
- Eklentiler için: Değişikliklerinizi kapsayan otomatik mocha testleri yazın.
- Örnekler: Gösterdiğiniz tüm işlevleri manuel olarak test edin.
- Codelabs için: Eğitimin tamamını temiz bir ortamda çalıştırın ve sağladığınız örnek kodları test edin.
Bilgi verin.
Bu, halkla ilişkiler oluşturmanın son ve muhtemelen en önemli kısmı olan özeti yazmaktır.
Mükemmel bir PR özeti yazmak, diğer geliştiricilerin değişikliklerinizi incelemesine yardımcı olarak değişikliklerinizin daha hızlı kabul edilme olasılığını artırır.
Özetiniz aşağıdakilere benzer bilgileri içermelidir:
- Halkla İlişkiler'inizin hangi sorunla ilgili olduğu.
- Halkla ilişkiler (PR) ile ilgili yaptığınız değişiklikler.
- Değişikliğinizi test etme yönteminiz.
- İnceleme uzmanlarının incelemesi istediğiniz her şey.
- İnceleme uzmanlarının ihtiyacı olduğunu düşündüğünüz diğer bilgiler.
Talebinizi oluştururken Halkla ilişkiler şablonuna
İyi Kodlamalar!