Gitlab kullanarak bir kod incelemesi nasıl kurulur?

Gitlab kullanarak kod incelemesi nasıl kurulur? Bunun Gitlab web sitesinde bir özellik olarak listelendiğini görüyorum, ancak nasıl kurulacağına dair talimatları bulamıyorum (Bu nedenle, Gitlab kullanım kılavuzuna herhangi bir bağlantı çok takdir edilecektir).

Araştırmalarımın bir kısmı, "İstekleri Birleştirme" nin gidilecek yol olduğunu gösterdi ... ancak bunları sınırlayıcı buluyorum. Yayınlanan bir birleştirme talebi, bir şube ile diğeri arasındaki tüm taahhütleri gösterir. Görünüşe göre her bir kayıt için oluşturulan farklılıkları görebiliyor gibiyim. Örneğin, gözden geçirmek istediğim bir dosyam olduğunu varsayalım. Bu yeni bir dosya ancak bir geliştirme dalında 10'un üzerinde değişiklik gönderdim. Bu dev dalı için entegrasyondan bir birleştirme isteği yayınlarsam, her biri dosyada yapılan artımlı değişiklikleri gösteren 10 işlem görüyorum ... Her şeyi gözden geçirmek istiyorum. Bu yeni!

Burada yanlış ağaca mı havlıyorum? GitLab'da kullanabileceğim gerçek bir kod inceleme aracı var mı, yoksa birleştirme istekleri doğru mu ve eğer öyleyse bunları yanlış mı kullanıyorum? Burada uygun bir kod incelemesi kurmanın en iyi yolu nedir?

Not: GitLab 6.4'ten beri, yan yana fark görünümü mevcuttur: bkz. " Çekme isteği 5308 ".

(Temmuz 2013)Her satıra henüz yorum yapma imkanı yoktur, sadece dosya seviyesinde.
Daniel Sokolowski bahseder yorumlarda o Başına hat yorumları şimdi desteklenir (09/2014):

Ekip üyeleriniz genel olarak birleştirme talebine veya satır yorumlarıyla belirli satırlara yorum yapabilir.

Bu yine de kod inceleme etkinliği için yardımcı olabilir.

6 yıl sonra GitLab 13.1 için (Haziran 2020) :

Birleştirme İsteği İncelemeleri Core'a taşındı

Başlangıçta GitLab 11.4'te GitLab Premium özelliği olarak sunulan Birleştirme İsteği İncelemeleri, birleştirme talebi gözden geçirenlerin şunları yapmasına olanak tanır:

• aynı anda birden fazla yorum gönderin,
• birleştirme isteği yazarı için bildirim sesini azaltmak ve
• daha uyumlu ve akıcı bir inceleme sürecine olanak tanır.

Piyasaya sunulduğundan beri , alıcı bazlı fiyatlandırma modelimizdeki yerini yeniden değerlendirdik ve 13.1'in bir parçası olarak bu özelliğin GitLab Core'a taşındığını duyurmaktan heyecan duyuyoruz.

Belgelere ve Soruna Bakın

Gitlab'da iki aydır neredeyse hiç sürtünme olmadan kod incelemeleri yapıyorum. Bir geliştirici her yeni taahhütte bulunduğunda e-posta bildirimleri göndermek için rss2email'i kurdum . Daha sonra, iletilen kod hakkında bazı yorumlar yapmak için kayıtlar için Gitlab'ın yorum özelliğini kullanıyorum.

Ne yazık ki Gitlab, dosyaların kendisi üzerinde yorum yapılmasına izin vermiyor, sadece kayıtlarda (aynen Github gibi, sanırım). Kendimi önceki bir işlemde kaçırdığım bir şeyi yorumlamam gereken bir durumda bulduğumda, yorumlanacak kod bölümünü getiren / değiştiren taahhüdü bulmak için suçlama aracını kullanıyorum.

Mükemmel olmaktan uzak ama şu ana kadar iyi çalışıyor.

Gönderilen kodu diğer depo için Birleştirme İsteği'nde veya mevcut depoda görebilirsiniz.
örnek http://demo.gitlab.com/diaspora/diaspora/commits/master

Ardından, kaydedilen dosya değişikliklerine (düğme Reply) veya tüm yürütmeye yorum ekleyebilirsiniz.

örnek http://demo.gitlab.com/diaspora/diaspora/commit/42f47626890218a180870bc3f44ec57625b0779c

Ortaya çıkan iletişim kod incelemesidir . Bununla birlikte, kişisel olarak, mümkün olduğunda yüz yüze iletişim ile tek bir bilgisayarda kod incelemesi yapmanızı ve sonuçları kaydetmek için veya daha fazla formalite gerektiğinde araçlar kullanmanızı tavsiye ederim.

Çok fazla taahhüt içeren bir dosya revue için, örneğin http://demo.gitlab.com/diaspora/diaspora/blame/master/README.mdblame kimin ne yaptığını anlamak için kullanarak buna bakın . Ancak bu görünümde iletişim kurma ve yorum ekleme seçeneği yoktur. Bu durumda sadece yorum olarak değişiklik eklemenizi tavsiye ederim.

Evet. Birleştirme istekleri, meslektaş incelemelerinin nasıl gerçekleştirildiğidir.

Tüm işlemlerin değişikliklerini gösteren bir 'fark' sekmesi olmalıdır (burada bahsedilmiştir: http://youtu.be/DyAX8ws5OIc?t=3m2s ).

Video aynı zamanda hakem incelemesinde nasıl kullanılabileceğini güzel bir şekilde açıklıyor.

Kod incelemelerinin normal kullanım durumu, ana veya benzeri ile birleştirmeden önce bir daldaki kodu incelemektir. Bir proje geliştirdiğim ve tüm kodun ekipteki herkes tarafından gözden geçirilmesini istediğim bir durum var.

Yaptığım şey şuydu:

İlk kaydı kontrol edin, üzerinde bir değişiklik yapın, taahhüt edin ve itin

git co -b FIRST_COMMIT eb67f06c2b3222c0219214b176c41922bc454881
vi README.md
git add README.md
git ci -m "First commit modified so can get full diff against it"
git push --set-upstream origin FIRST-COMMIT

Son kaydı kontrol edin, üzerinde bir değişiklik yapın, taahhüt edin ve itin

git co -b master
vi README.md
git add README.md
git ci -m "Last commit modified so can get full diff against it"
git push --set-upstream origin LAST-COMMIT

GitLab / GitHub'da bir çekme isteği oluşturun

• LAST_COMMIT ile FIRST_COMMIT arasında bir birleşiyor

Benim için çalışıyor!