Çarpma
HackerOne’da Spotify Mühendislik Çerçevesini kullanıyoruz, yani kadrolarda çalışıyoruz. Her takım, şirket içindeki küçük bir başlangıç gibi belirli görevlere odaklanan mini bir takımdır.
Bir ekibin özü, ortak bağlamlarında ve alışkanlıklarında yatmaktadır. Bir kod incelemesine katılırken, birleştirildiğinde yazılım kalitesi ve ekibin kimliği üzerinde önemli bir etkisi olan birkaç küçük karar alınmalıdır.
Sadece neyin yanlış olduğunu değil, nedenini de açıklamak çok önemlidir. Sadece sorunu işaret ederken, düzeltilebilir, ancak temel sorunu anlamadan sorunun gelecekte tekrarlanmasına yol açacaktır. Kapsamlı açıklamalar sağlamak için zaman ve çaba harcamanın bir bileşik etkisi vardır. Yazarın yeni bir deneyim kazanmasına yardımcı olur ve en iyi uygulamalar hakkında bir tartışmaya yol açabilir, bu da paylaşılan bilgi ve takım büyümesine yol açar. Bu şekilde gözden geçirmediğiniz Bayan hala geri bildiriminizden yararlanacaktır.
Neye benzediğine veya alternatif çözümleri ve ödünleşimlerini sunmanın örneklerini sağlamak da aynı derecede önemlidir. Bunu yapmak, geri bildirim yerine besleme prensibine benzer şekilde bir öğrenme ve iyileştirme kültürünün geliştirilmesine yardımcı olur. Bilgi paylaşmaya ve standartlar oluşturmak için yatırım yapmak, uzun vadede zaman ve çaba tasarrufu sağlayarak ekibin daha gelişmiş görevlere odaklanmasını sağlayabilir.
Kod incelemeleri sırasında düşüncelerinizi ortaya çıkarmak için zaman ayırmak, özellikle iletişimin kolayca yanlış yorumlanabileceği eşzamanlı ortamlarda, ekip içinde güven ve uyum oluşturur. Sözlerinizin tonuna ve yazar üzerindeki potansiyel etkisine dikkat etmek önemlidir. Bir yorum bırakmadan önce kendinize neyi başarmayı umduğunuzu sorun ve olası etkileri dikkate alın. Eğer amaç bilginizi sergilemek veya egonuzu şişirmekse, sonuçta bir bağ fırsatı ekibin içindeki bir yarık haline dönüştürecek. Kötü bir örnek “Bunu buna değiştirebilir misin?” Veya “Test yazmadın” – bu çok zorlu. Bunun yerine, “Ya bunu yapmaya çalışırsak?” Veya “Bunu düşündün mü?”
Son olarak, paylaşım iki yönlü bir sokaktır. Kod incelemeleri sadece hata bulmak değil, aynı zamanda bir öğretim anı ile ilgilidir. Bilginizi güncel tutmanıza yardımcı olurlar, ancak herhangi bir sorunu kaçırmamanızı sağlarlar. Sadece meraktan çıksa bile, anlamadığınız kod hakkında daha fazla bilgi istemekten çekinmeyin ve birlikte çalıştığınız teknolojiyle ilgili belgeleri okuduğunuzdan emin olun. Ayrıca öğrenirken, kişiye takdir göstermek iyidir. “Burada bu yeniden düzenlemeyi gerçekten seviyorum! Bunu değiştirdiğiniz için teşekkürler! ” Ve “vay, bunu bu şekilde kullanabileceğimizi bilmiyordum” uzun bir yol kat edebilir.
Yürütme – Teslimat için optimize edin
Takım olarak büyümek harika, ancak üretimde yazılım olan bir ekip olmak daha büyük. Kod incelemesinin ilk amacı, kaliteli bir çözümün dağıtılmasıdır. İşte bunu yapmanın bazı yolları:
- Bağlam içinde gözden geçirin: Eldeki belirli sorunu ele aldığınızdan emin olun ve kapsam sürünmesine izin vermeyin. Basit bir kopya değişikliği için ilgisiz dosyalarda kapsamlı yeniden düzenleme değişiklikleri yapmaktan veya özellik bayraklarının kaldırılmasıyla ilgili tartışmalara başlayın.
- Bir takip dosyası: Önemli sorunlarla karşılaşırsanız, onları GitLab gibi bir yerde not edin, böylece buna göre anlaşılabilir ve önceliklendirilebilirler.
- Yorumunuz engelliyorsa veya değilse açıkça iletişim kurun: Yanlış anlamalar meydana gelebilir, bu nedenle birleştirmeden önce bir yorumun ele alınması gerekip gerekmediğini veya gelecekteki iyileştirmeler için bir öneri olup olmadığını bildirmek önemlidir.
- Savaşlarınızı seçin: Çözümün mükemmel olması gerekmediğini unutmayın. Bazen afetleri önlemek için korkuluklar sağlamaya odaklanmak, mutlak mükemmellik elde etmeye çalışmaktan daha pratiktir.
- Mrs’i birden çok parçaya ayırın: Bu yaklaşım, incelemeyi kolaylaştırır ve tüm süreci tutmasını önler.
- Sola kaydırma: Önemli tasarım kararları konusunda meslektaşlarınızla eşleştirin ve bunları gevşeklik üzerine aşırı iletişim kurun. Bu yaklaşım, büyük değişiklikleri riske atmaya ve son dakika sürprizlerinden veya sorunlardan kaçınmaya yardımcı olur.
Etkili kod incelemesi için ipuçları
- Katmanlar halinde inceleme: Entteation gibi küçük detaylar hakkında endişelenmeden önce, özelliğin tüm kullanım durumlarını karşılamasını sağlamak gibi en önemli konulara odaklanın.
- Parçalarda inceleme: Dikkatinizin solmasını önlemek ve incelemenizin kalitesini korumak için sorunları kırın ve istekleri daha küçük parçalar halinde birleştirin.
- Demolar ve ekran görüntüleri isteyin: Her çözümü kendiniz manuel olarak test etmek zorunda değilsiniz. Akranlarınıza güvenin, ancak yüksek bahisli sorunlar veya potansiyel kenar durumları için bir demo veya ekran görüntüsü isteyin.
- Duraklatmaktan korkmayın: Bir MR’yi onaylamakta tereddüt ediyorsanız, sorunun amacını ve potansiyel riskleri gözden geçirmek için bir dakikanızı ayırın. Acele etmekten acele etmekten daha iyi bir inceleme yapmak daha iyidir.
- Müzakere edilebilir olduğunuzu gösterin: Gerçekten yeniden düzenlenmesi gerektiğini düşündüğünüz bir şey olduğunu varsayalım. Tartışmayı germek yardımcı olmaz. “Bu özelliğin yakında yayınlanması gerektiğini biliyorum, bu yüzden bu değişiklik bu MR için çok fazla olabilir, ancak bunun için bir Gitlab bileti oluşturmak ve yaklaşan sprintlerden birinde almak mümkün mü?”
- Örnekleri kullanın: Bu programlama dilinin resmi belgelerinden örnekler bulmaya çalışın. Bu gerçekten iyi bir alışkanlık ve insanları aynı şeyi yapmaya teşvik ediyor.
- Bir kontrol listesi tutun: Ekibinizdeki tekrar eden sorunları takip edin ve incelemelerinize öncelik verin. İkinci doğa olmalarına yardımcı olmak için bunları MR şablonunuza eklemeyi düşünün.
- İncelemelere öncelik verin: Birleştirme istekleri değerlidir ve hızlı bir geri bildirim döngüsünü korumak ve sprintteki gecikmelerden kaçınmak için derhal gözden geçirilmelidir. Çok fazla açık çalışma birleştirildiğinde sorunlara neden olabilir, bu nedenle incelemelerinize göre öncelik verin.
Kod incelemeleri, yüksek kaliteli yazılım çözümleri sağlamak ve kuruluşunuzun teknik vizyonunu güçlendirmek için çok önemlidir. Unutmayın, tüm bu uygulamaları aynı anda uygulamak zorunda değilsiniz. Sizinle yankılanan bir çift seçin ve oradan başlayın. Ve akranlarınızdan girdi almayı unutmayın. Bir takım için işe yarayan başka bir ekip için işe yaramayabilir, bu yüzden sizin için en iyi olanı bulmak önemlidir.