it-swarm.dev

كيفية إعداد مراجعة الكود باستخدام Gitlab؟

كيف يمكن للمرء إعداد مراجعة الكود باستخدام Gitlab؟ أرى أنه مدرج كميزة على موقع Gitlab على الويب ، لكن لا يمكنني العثور على إرشادات حول كيفية إعداد واحد (في هذا الشأن ، سيكون أي رابط لدليل مستخدم Gitlab موضع تقدير كبير).

لقد أوضحت بعض عمليات البحث التي أجريتها أن "دمج الطلبات" هي الطريق للذهاب ... ولكني أجدها محدودة. يُظهر طلب دمج تم إصداره كافة الالتزامات بين فرع واحد والآخر. يبدو لي أن أكون قادرًا فقط على عرض الفرق الذي تم إنشاؤه لكل التزام فردي. على سبيل المثال ، دعنا نقول أن لدي ملف أريد مراجعته. إنه ملف جديد لكنني قدمت تغييرات عليه أكثر من 10 إلتزامات على فرع مطور. إذا قمت بإصدار طلب دمج لهذا الفرع ديف من التكامل أرى 10 يرتكب كل منها إظهار التغييرات المتزايدة التي تم إجراؤها على الملف ... أريد مراجعة الأمر بالكامل. إنه جديد!

هل أنا ينبح الشجرة الخطأ هنا؟ هل هناك أداة حقيقية لمراجعة الشفرة يمكنني استخدامها في GitLab ، أم أن طلبات الدمج هي الطريقة التي يجب اتباعها ، وإذا كانوا يستخدمونها بشكل غير صحيح؟ ما هي أفضل طريقة لإعداد مراجعة رمز مناسبة هنا؟

82
djc6535

ملاحظة: منذ GitLab 6.4 ، عرض فرق جنبًا إلى جنب متاح: راجع " طلب السحب 5308 ".

(يوليو 2013)لا توجد إمكانية حتى الآن للتعليق على كل سطر ، على مستوى الملف فقط.
دانيال سوكولوفسكي ذكر في التعليقات thatالآن يتم دعم التعليقات لكل سطر(09/2014):

يمكن لأعضاء فريقك التعليق على طلب الدمج بشكل عام أو على أسطر محددة مع تعليقات الأسطر.

لا يزال يمكن أن تساعد في نشاط مراجعة التعليمات البرمجية.

https://f.cloud.github.com/assets/4224518/1558702/e0fe633a-4fa3-11e3-9388-3f3e445cb6d4.png

23
VonC

لقد تم القيام بمراجعات الكود في Gitlab لأكثر من شهرين دون أي احتكاك تقريبًا. لقد قمت بإعداد rss2email لإرسال إشعارات عبر البريد الإلكتروني في كل مرة يقوم فيها مطور بدفع التزامات جديدة. ثم أستخدم ميزة التعليقات في Gitlab لالتزامات لتقديم بعض التعليقات حول الكود المدفوع.

لسوء الحظ ، Gitlab لا يسمح بالتعليقات على الملفات نفسها ، فقط في عمليات ارتكاب (مثل Github ، أعتقد). كلما وجدت نفسي في موقف أحتاج إلى التعليق على شيء فاتني في التزام سابق ، أستخدم أداة اللوم للعثور على الالتزام الذي أدخل/غير قسم الشفرة للتعليق عليه.

إنه أبعد ما يكون عن الكمال ، لكنه يعمل بشكل جيد حتى الآن.

8
Herberth Amaral

يمكنك مشاهدة التعليمات البرمجية المرسلة في "دمج طلب" لمستودع آخر أو في مستودع الحالي.
مثال http://demo.gitlab.com/diaspora/diaspora/commits/master

بعد ذلك يمكنك إضافة تعليقات على تغييرات الملفات الملتزمة (زر Reply) أو إلى الالتزام كله

مثال http://demo.gitlab.com/diaspora/diaspora/commit/42f47626890218a180870bc3f44ec57625b0779c

الناتج التواصل هو مراجعة الكود . ومع ذلك ، أوصي شخصيًا بإجراء مراجعة الكود على جهاز كمبيوتر واحد مع التواصل المباشر وجهًا لوجه كلما أمكن ذلك ، واستخدام الأدوات لتسجيل النتائج أو عند الحاجة إلى مزيد من الإجراءات الشكلية.

بالنسبة للملف الذي يحتوي على الكثير من الإلتزامات ، على سبيل المثال http://demo.gitlab.com/diaspora/diaspora/blame/master/README.md انظر إليها باستخدام blame لفهم من فعل ما. ولكن في طريقة العرض هذه ، لا يوجد خيار للتواصل وإضافة التعليقات. أوصي فقط بإضافة التغييرات كتعليقات في هذه الحالة.

3
Paul Verest

نعم فعلا. طلبات الدمج هي كيفية تحقيق المراجعات النظيرة.

يجب أن يكون هناك علامة تبويب "فرق" تعرض التغييرات لجميع الإلتزامات (المذكورة هنا: http://youtu.be/DyAX8ws5OIc؟t=3m2s ).

يوضح الفيديو أيضًا كيف يمكن استخدامه لمراجعة النظراء.

0
onionjake

الاستخدام العادي لحالة مراجعة الكود هو مراجعة الكود على الفرع قبل الاندماج في برنامج الماجستير أو ما شابه. لدي موقف حيث قمت بتطوير مشروع وأريد أن تتم مراجعة جميع التعليمات البرمجية من قبل الجميع في الفريق.

ما فعلته هو:

تحقق من الالتزام الأول ، قم بإجراء تغيير عليه ، الالتزام والدفع

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

تحقق من الالتزام الأخير ، قم بإجراء تغيير عليه ، الالتزام والدفع

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 ، قم بإنشاء طلب سحب

  • إنه دمج واحد من LAST_COMMIT إلى FIRST_COMMIT

يعمل لدي!

0
HankCa