Gitlab का उपयोग करके कोड समीक्षा कैसे सेट करें?


85

Gitlab का उपयोग करके कोई कोड समीक्षा कैसे सेट करता है? मैं इसे Gitlab वेबसाइट पर एक विशेषता के रूप में सूचीबद्ध देखता हूं, लेकिन मैं यह कैसे सेट अप कर सकता हूं, इसके लिए निर्देश नहीं पा सकते हैं (उस मामले के लिए, Gitlab उपयोगकर्ता पुस्तिका के किसी भी लिंक को सबसे अधिक सराहा जाएगा)।

मेरी खोज में से कुछ ने संकेत दिया है कि 'मर्ज अनुरोध' जाने का तरीका है ... लेकिन मैं उन्हें सीमित कर रहा हूं। एक जारी किया गया मर्ज अनुरोध एक शाखा और दूसरे के बीच सभी कमिटों को दिखाता है। मुझे लगता है कि केवल प्रत्येक व्यक्ति के लिए उत्पन्न अंतर को देखने में सक्षम होना चाहिए। उदाहरण के लिए, मान लें कि मेरे पास एक फ़ाइल है जिसकी मैं समीक्षा करना चाहता हूं। यह एक नई फ़ाइल है, लेकिन मैंने 10 से अधिक बदलावों को देव शाखा पर प्रस्तुत कर दिया है। यदि मैं एकीकरण से उस देव शाखा के लिए एक मर्ज अनुरोध जारी करता हूं, तो मैं देखता हूं कि प्रत्येक 10 हिट करता है जो फ़ाइल में किए गए वृद्धिशील परिवर्तन को दिखाता है ... मैं पूरी बात की समीक्षा करना चाहता हूं। यह नया है!

क्या मैं यहाँ गलत पेड़ को काट रहा हूँ? क्या एक वास्तविक कोड-समीक्षा उपकरण है जिसे मैं GitLab में उपयोग कर सकता हूं, या मर्ज करने के तरीके का अनुरोध कर रहा हूं, और यदि वे हैं तो क्या मैं उनका गलत तरीके से उपयोग कर रहा हूं? यहां एक उचित कोड समीक्षा सेट करने का सबसे अच्छा तरीका क्या है?


1
GitLab 6.4 और इसके साथ-साथ अलग-अलग दृश्य कोड समीक्षा के लिए मदद कर सकते हैं: नीचे मेरा जवाब देखें
VonC

GitLab 13.1 (जून 2020) के साथ, अब आपके पास मर्ज अनुरोध समीक्षाएं हैं। देखें नीचे मेरी संपादित जवाब
VonC

जवाबों:


24

नोट: GitLab 6.4 के बाद से, साइड-बाय-साइड अंतर दृश्य उपलब्ध है: " पुल अनुरोध 5308 " देखें।

(जुलाई 2013)प्रत्येक पंक्ति पर टिप्पणी करने की अभी तक कोई संभावना नहीं है, केवल फ़ाइल स्तर पर।
डैनियल सोकोलोव्स्की ने टिप्पणियों में उल्लेख किया है कि प्रति पंक्ति टिप्पणियाँ अब समर्थित हैं (09/2014):

आपकी टीम के सदस्य सामान्य रूप से या विशिष्ट टिप्पणियों के साथ लाइन टिप्पणियों के साथ मर्ज अनुरोध पर टिप्पणी कर सकते हैं।

वह अभी भी कोड समीक्षा गतिविधि के लिए मदद कर सकता है।

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


6 साल बाद, गीताब 13.1 (जून 2020) के लिए :

मर्ज अनुरोध समीक्षा कोर में ले जाया गया

मूल रूप से GitLab प्रीमियम सुविधा के रूप में GitLab 11.4 में पेश किया गया, मर्ज अनुरोध समीक्षा में मर्ज अनुरोध समीक्षकों को अनुमति दें:

  • एक साथ कई टिप्पणियाँ सबमिट करें,
  • मर्ज अनुरोध लेखक के लिए अधिसूचना शोर में कटौती, और
  • अधिक सामंजस्यपूर्ण और सुव्यवस्थित समीक्षा प्रक्रिया की अनुमति।

https://about.gitlab.com/images/13_1/batch_comments.png

इसकी शुरुआत के बाद से, हमने अपने खरीदार-आधारित मूल्य निर्धारण मॉडल में इसके स्थान का पुनर्मूल्यांकन किया है और 13.1 के भाग के रूप में हम घोषणा करने के लिए उत्साहित हैं कि यह सुविधा अब गिटलैब कोर में स्थानांतरित हो गई है।

प्रलेखन और मुद्दा देखें


प्रति पंक्ति टिप्पणियों का अब समर्थन किया जाता है: "आपकी टीम के सदस्य सामान्य रूप से मर्ज अनुरोध पर या विशिष्ट टिप्पणियों के साथ लाइन टिप्पणियों पर टिप्पणी कर सकते हैं।" ( about.gitlab.com/2014/09/29/gitlab-flow )
डैनियल

1
@DanielSokolowski बढ़िया! अधिक दृश्यता के जवाब में मैंने आपकी टिप्पणी को शामिल किया है।
वॉनक

9

मैं लगभग दो महीने से Gitlab में लगभग कोई घर्षण के साथ कोड समीक्षा कर रहा हूं। मैंने ईमेल सूचनाएँ भेजने के लिए rss2email सेटअप किया है, हर बार एक डेवलपर नए कमिट को धकेलता है। फिर मैं पुश कोड के बारे में कुछ टिप्पणी करने के लिए कमिट के लिए Gitlab के कमेंट फीचर का उपयोग करता हूं।

दुर्भाग्य से, गीतालाब फाइलों पर टिप्पणियों की अनुमति नहीं देता है, केवल कमिट्स में (गितुब की तरह, मुझे लगता है)। जब भी मैं खुद को ऐसी स्थिति में पाता हूं कि मुझे किसी ऐसी टिप्पणी पर टिप्पणी करने की आवश्यकता होती है जिसे मैंने पिछली प्रतिबद्ध में याद किया है, तो मैं टिप्पणी करने के लिए कोड अनुभाग को प्रस्तुत / परिवर्तित करने के लिए दोष उपकरण का उपयोग करता हूं।

यह एकदम सही है, लेकिन यह अब तक अच्छा काम कर रहा है।


1
Rss2email के बजाय कोई भी धक्का देने के बारे में सूचित करने के लिए Gitlab सूचनाओं का उपयोग कर सकता है।
वादीप

मुझे एक ही समस्या है / वर्कअराउंड। मेरा मानना ​​है कि यह एक अच्छा फीचर एडिशन होगा, जिसमें आप डिफरेंट या फाइल व्यू में किसी खास लाइन को सही ठहराने के लिए कमेंट जोड़ सकते हैं (मेरा मतलब वेब इंटरफेस ब्राउजिंग फाइल्स से है या डिफरेंट है, रनिंग ब्लेम नहीं)।
अलेजांद्रोवडी

2

आप अन्य अनुरोधों के लिए या वर्तमान भंडार में मर्ज अनुरोध में प्रस्तुत कोड देख सकते हैं।
उदाहरण 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समझने के लिए उपयोग करके देखें कि किसने क्या किया। हालाँकि इस दृश्य में टिप्पणियों को जोड़ने और जोड़ने का कोई विकल्प नहीं है। मैं इस मामले में टिप्पणियों के रूप में केवल परिवर्तन जोड़ने की सलाह दूंगा।


7
मुझे आपके उत्तर में पहले, दूसरे और अंतिम लिंक के लिए 404 मिलता है।
ब्रायन ओकले

1
जैसा कि होम पेज पर कहा गया है, Demo.gitlab.com "IS A SANDBOX - यह हर घंटे रीसेट हो जाता है" इसलिए सभी उदाहरण मिटा दिए गए। यह उदाहरणों के लिए एक अच्छा वाहन नहीं है।
यूरिया ब्लेथरविक

हां, कृपया उचित उदाहरणों के साथ इस पर पुनर्विचार करें। आपका जवाब आम तौर पर ठोस सलाह लगता है।
डेटा

0

हाँ। मर्ज अनुरोध हैं कि सहकर्मी की समीक्षा कैसे पूरी की जाती है।

एक 'भिन्न' टैब होना चाहिए जो सभी कमिट के परिवर्तन (यहाँ उल्लिखित है: http://youtu.be/DyAX8ws5OIc?t=3m2s ) दिखाएगा ।

वीडियो भी अच्छी तरह से बताता है कि यह कैसे सहकर्मी की समीक्षा के लिए इस्तेमाल किया जा सकता है।


0

कोड समीक्षाओं का सामान्य उपयोग-मामला मास्टर या समान में विलय करने से पहले एक शाखा पर कोड की समीक्षा करना है। मेरे पास एक ऐसी स्थिति है जहां मैंने एक परियोजना विकसित की है और चाहता हूं कि सभी कोड टीम द्वारा सभी की समीक्षा की जाए।

मैंने जो किया वह था:

पहले कमिट को चेकआउट करें, उसमें बदलाव करें, कमिट करें और पुश करें

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 तक विलय हो रहा है

मेरे लिये कार्य करता है!


क्या यह आपको रिपॉजिटरी में दो "जंक" शाखाओं के साथ नहीं छोड़ता है, और मास्टर शाखा में टिप्पणियों का कोई ट्रैकिंग नहीं है? यदि टिप्पणियों में कोड परिवर्तन की आवश्यकता होती है, तो क्या आप उन्हें मास्टर में विलय करते हैं?
user2084572

हाँ FIRST_COMMIT और LAST_COMMIT शाखाएँ होंगी जिन्हें हटाना आसान है ( git br --delete --force origin FIRST_COMMIT LAST_COMMIT; git br --delete --force FIRST_COMMIT LAST_COMMIT)। आप उस परिवर्तन में, या मैन्युअल रूप से अलग-अलग समस्याएँ पैदा करने के लिए मास्टर से अलग शाखा का उपयोग कर सकते हैं। और बाद में बहुत अधिक प्रतिक्रिया होने पर एक या अधिक शाखाएँ (जैसे एक मुद्दा) बनाएं।
हांकका
हमारी साइट का प्रयोग करके, आप स्वीकार करते हैं कि आपने हमारी Cookie Policy और निजता नीति को पढ़ और समझा लिया है।
Licensed under cc by-sa 3.0 with attribution required.