Her zaman iyi bir ipucunun ayırt edici özelliklerinden birinin, her geliştirme döngüsü sırasında gerektiği gibi ek eğitim veren biri olduğunu hissettim. Benim için, bu sadece kendimi kodlamıyorum ya da kodları gözden geçirmiyorum, aynı zamanda daha genç geliştiricilerle birlikte oturduğum, üzerine bastığım kara mayınlarından kaçınmalarına yardımcı olmak için onlarla programlamayı eşleştiriyorum.
Temel olarak, birincil hedefimizin eğitim olduğu konusunda hiçbir yanılsamam yok - değil. İster kıdemli, ister potansiyel müşteri, ister küçük bir geliştirici olun, hedef sizin düzenlemeniz değildir. Amaç her zaman müşteriye kalite kodu sunmaktır. Tercihen zamanında, bütçede, istediklerini yapmak. Bununla birlikte, tüm işleri kendim yapmamın imkansız olduğunu biliyorum, bu yüzden ekibin başarılı olmasına yardımcı olmak için bana bir ipucu olarak görev yaptım. Ve bu, doğada ortaya çıktıklarında eğitim fırsatlarını tanımak anlamına gelir.
Bu nedenle, çekme taleplerinin gençleri eğitmek için yer olup olmadığı hakkındaki sorunuzda, bunlar sırasında öğrenilebilir anların ortaya çıkmasının nadir olmadığını söylemeliyim. Hey, ilk birleşme çatışmanızla uğraşmanız gerekecek, gözden geçirmeden sonra bunu ele alalım. Oh bak, DAO'nuz için birim testleri eklemediniz, incelemenizi bu yeni yöntemleri kapsamak için bir şansımız olana kadar erteleyeceğiz. Neden bu finansal hesaplamada çift temel öğeleri kullanmanın BigDecimals'den daha iyi olacağını düşündünüz? Evet, bu çok nadir değil.
Yani, kesinlikle olabileceğini söylesem de, açıkça bir çekme talebinin ana amacı bu değil. Ayrıca bir çekme isteğindeki kodun üretime hazır olduğuna dair bir beklenti olduğunu düşünmüyorum. Genellikle değil.
Özellik ve sürüm dallarını gitflow stili dallanma stratejisinde kullanıyorsanız, çekme istekleriniz daha çok sürüm adaylarına benziyor. Üretime hazır değil, daha yaklaşan bir şey. Kodunuzun derlendiğini biliyorsunuz (doğru) ve kullanıcı hikayesinin amaçlarına uygun olduğunu iddia etmek için yeterli test covfefe var. Ve geliştirme ortamınızda zaten birkaç entegrasyon testi yaptığınız için, PR'nizi incelerken, değişikliklerinizi göstermeniz istenirse, gitmeye hazır harika bir demonuz var.
Nihayetinde, PR incelemeleri sırasında yardım sağlamamız gerektiğini hissediyorum, ancak bu yardım genel kodlama ile ilgili değil. Bunun yerine, önerilen kodun üretim-kalite kodunun bir çalışma tabanına dahil edilmesi için hazırlanmasıyla ilişkilidir. Halkla İlişkiler, geliştiricilere Halkla İlişkiler'e dahil ettikleri her değişiklik için bir gerekçe ve sağlam bir kavrayışa sahip olduklarını gösterme fırsatıdır. Ve o zaman bile - onları birim testler, demolar ve birçok soru ile tarttıktan sonra bile - önerilen değişikliklerin üretime hazır olacağına dair hiçbir beklentimiz yok.
Tüm bunlardan sonra kod yakın. Ama sonra KG'ye işkence etmesine izin verdik.