GitHub पुल अनुरोधों पर सहकर्मी समीक्षा कैसे करें?


12

हम Bitbucket से GitHub की ओर बढ़ रहे हैं और एक चीज़ जो हम संघर्ष कर रहे हैं, वह सहकर्मी कोड समीक्षाएं हैं जो इस तरह से Bitbucket पर बहुत आसानी से काम करती हैं:

  1. लेखक ने एक पुल अनुरोध (GitHub: वही) खोला
  2. लेखक समीक्षक के रूप में उसकी / उसके सहयोगियों जोड़ा (GitHub: ?? साथ यहां संघर्ष कर कई असाइनी)
  3. समीक्षक या तो:
    1. हरे रंग के चेक मार्क के साथ पीआर को मंजूरी दी (GitHub: ??)
    2. जोड़े गए टिप्पणियाँ (GitHub: वही)
    3. हल्के कार्य किए गए (GitHub: इसी तरह की अगर - [ ]सिंटैक्स PR विवरण में उपयोग किया जाता है, तो शर्म की बात है कि यह काम नहीं करता है)
  4. वहाँ पीआर की एक सूची है जहाँ मैं एक नज़र में देख सकता हूँ जिसकी समीक्षा की गई है और विलय करने के लिए ठीक है और जिसे और ध्यान देने की आवश्यकता है (GitHub: ??)

मुझे यह इंगित करना चाहिए कि यदि संभव हो तो हम तीसरे पक्ष के कोड समीक्षा टूल से बचना चाहते हैं और किसी प्रकार के वर्कआयर्ड के साथ वेनिला गिटहब पर रहना चाहेंगे।


1
लगता है कि आप समय से पहले स्विच कर सकते हैं। वैसे भी क्यों स्विच करें, खासकर अगर नई चीज़ में वे सभी सुविधाएँ न हों जिनकी आपको आवश्यकता है?
नानी

अपने prq पर एक टिप्पणी लिखें, और आप जो भी अधिसूचना प्राप्त करना चाहते हैं, उस पर प्रकाश डालें। समीक्षक अपनी समीक्षा राय दिखाने के लिए टैग जोड़ सकते हैं।
विल्बर्ट

यह भी ध्यान दें कि आप एक पूरी टीम @ @ MyCompany / Developers
Whymarrh

जवाबों:


6

मैंने जो कुछ देखा है, उनमें से अधिकांश कदम सम्मेलन द्वारा जीथब पर किए गए हैं, न कि किसी आधिकारिक जीथब द्वारा प्रदान की गई प्रक्रिया द्वारा।

मेरे नियोक्ता गितुब का उपयोग करते हैं, मैं अच्छी संख्या में छोटे स्रोत परियोजनाओं को चलाता हूं, और अन्य खुले स्रोत परियोजनाओं में कभी-कभी योगदान देता हूं।

यहाँ बताया गया है कि मैंने आमतौर पर इसे कैसे किया है:

लेखक समीक्षकों के रूप में अपने सहयोगियों को जोड़ रहा है:

यह परियोजना से परियोजना के लिए भिन्न होता है, लेकिन सामान्य तौर पर, निर्दिष्ट सहकर्मी समीक्षक परियोजना के सभी योगदानकर्ता होते हैं

ओपन सोर्स प्रोजेक्ट्स से लगता है कि एक मोटा पदानुक्रम है - हो सकता है कि एक "कोर" योगदानकर्ता द्वारा ठीक दिए जाने के बाद उनका सम्मेलन केवल विलय करने के लिए हो।

जिस दुकान पर मैं वर्तमान में कार्यरत हूं, हम टीम में शामिल आधा दर्जन डेवलपर्स में से किसी एक के बाद मर्ज कर देते हैं।

दुर्लभ अवसरों पर टीम में कोई व्यक्ति किसी अन्य डेवलपर को विशेष रूप से कॉल करने के लिए एक टिप्पणी का उपयोग कर सकता है जो उन्हें लगता है कि विलय होने से पहले सहकर्मी की समीक्षा करना चाहिए, लेकिन अन्यथा, जो कोई भी पहले वहां पहुंचता है और ऐसा लगता है कि ऐसा करने से वह समीक्षा कर सकता है और टिप्पणी कर सकता है।

समीक्षक अनुमोदन:

अनुमोदन आमतौर पर "+1" या "lgtm" कहकर पुल अनुरोध पर एक टिप्पणी करके दिखाया जाता है (मुझे अच्छा लगता है)।

हल्के कार्य:

मैंने चेकबॉक्स का भी उपयोग किया है, लेकिन ज्यादातर मामलों में, एक पुल अनुरोध पर प्रत्येक टिप्पणी को एक अंतर्निहित "कार्य" माना जाता है जिसे निम्न द्वारा हल किया जाता है:

  • उस कोड को बदलना जिस पर लाइन टिप्पणी कर रही है
  • एक और टिप्पणी के साथ प्रतिक्रिया

एक नज़र में देखना कि क्या मंजूर है और क्या अभी भी समीक्षा की आवश्यकता है:

मैंने क्रोम के लिए लुक्स गुड टू मी एक्सटेंशन का उपयोग किया है , जो आपको पुल अनुरोध स्क्रीन से ऐसा दृश्य देता है। पुल अनुरोध सूची दृश्य हाल ही में Github परिवर्तनों से टूट गया है, हालांकि लगता है।

हमारी साइट का प्रयोग करके, आप स्वीकार करते हैं कि आपने हमारी Cookie Policy और निजता नीति को पढ़ और समझा लिया है।
Licensed under cc by-sa 3.0 with attribution required.