2016-12-11 7 views
74

एक विशेषता दिसंबर, 7, 2016 पर कहा, GitHub ब्लॉग पर घोषणा, विकल्प to add reviewers to a Pull Requestगिटहब पर, समीक्षक और असाइन करने वाले के बीच क्या अंतर है?

GitHub Review Option

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

आप उन लोगों की एक सूची भी देख सकते हैं जिन्हें आप पुल अनुरोध पृष्ठ साइडबार में समीक्षा की प्रतीक्षा कर रहे हैं, साथ ही उन लोगों की समीक्षा की स्थिति भी देख चुके हैं जो पहले से ही उन्हें छोड़ चुके हैं।

हालांकि, स्पष्ट एक पीआर के लिए एक समीक्षक पहले से ही बताए लोग (असाइनी विकल्प) द्वारा किया गया था सेटिंग।

अब दोनों विकल्पों के साथ, प्रत्येक विकल्प की भूमिका क्या है क्योंकि वे दोनों एक ही अंतिम लक्ष्य साझा करते हैं?

उत्तर

63

संपादित करें:

कई ओएसएस देखरेख, समीक्षक क्या शब्द होना चाहिए के रूप में परिभाषित किया गया है के साथ चर्चा के बाद: समीक्षा करने के लिए (किसी के कोड) और एक हारने definiton नीचे समझाया "समनुदेशिती" है ।

"समीक्षक" के लिए: कोई आपको कोड की समीक्षा करना चाहते हैं। अनिवार्य रूप से उस क्षेत्र के लिए ज़िम्मेदार व्यक्ति या प्रतिबद्धता विलय के लिए ज़िम्मेदार नहीं है। गिटहब ऑटो-सुझाव के रूप में, जो पहले कोड के उस हिस्से पर काम कर सकता है, हो सकता है।

"असाइनर" के लिए: परियोजना की टीम/रखरखाव तक इसका क्या अर्थ है और कोई सख्त परिभाषा नहीं है। यह पीआर ओपनर हो सकता है, या उस क्षेत्र के लिए जिम्मेदार कोई व्यक्ति (जो समीक्षा के बाद पीआर स्वीकार करने जा रहा है या इसे बंद कर सकता है)। यह गिटहब तक नहीं है कि यह परिभाषित करने के लिए कि यह परियोजना प्रोजेक्टर के लिए क्या खुला है, जो उनके प्रोजेक्ट के लिए सबसे अच्छा है।

पिछला जवाब:

ठीक है, मैं आगे जाना है और अपने ही सवाल का जवाब देंगे।

लिखने का उपयोग के साथ उपयोगकर्ताओं की पीआर के लिए: अनुदिष्ट एक ही व्यक्ति है जो पीआर खोला होगा, और समीक्षक वर्ष समनुदेशिती समारोह (की समीक्षा कोड) की जगह लेंगे, समनुदेशिती पसंद के इस एक कोई जा रहा है।

लिखने-उपयोग के बिना उपयोगकर्ताओं के पीआर के लिए (बाहर योगदानकर्ताओं): लिखने का उपयोग के साथ किसी ने खुद को (या अन्य लिखने विशेषाधिकार सदस्य) आवंटित होगा, पीआर (समीक्षक) की समीक्षा करें। Assignee खाली है।

बाहरी योगदानकर्ताओं से अधूरा पीआर के लिए: लेखन-पहुंच सदस्य अधूरा काम करेगा और उसके लिए असाइन करेगा। Assignee होने के नाते, वह कार्य को पूरा करने के लिए जिम्मेदार होगी। चूंकि पीआरएस का मुख्य कारण परिवर्तनों की समीक्षा कर रहा है, इसलिए वह परिवर्तनों की समीक्षा करने के लिए कुछ अन्य लोगों का चयन करेगी।

+6

प्रत्येक नए टीम के सदस्य के लिए मुझे यह जवाब देने के लिए एक लिंक भेजना चाहिए कि असाइनियों और समीक्षकों से कैसे निपटें। जो मुझे इस विचार के लिए प्रेरित करता है कि यहां कुछ मौलिक रूप से गलत है :( –

12

गिटहब में एक समीक्षक एक व्यक्ति है जो पुल अनुरोध की समीक्षा करता है। एक प्रोजेक्ट स्वामी किसी भी रखरखाव से समीक्षा का अनुरोध कर सकता है, वे एक विकल्प भी सेट कर सकते हैं ताकि पुल अनुरोध केवल तभी विलय किया जा सके जब इसे लिखने वाले पहुंच वाले रखरखाव में से एक द्वारा समीक्षा की जा सके।

आधिकारिक github documentation के अनुसार, Assignee एक व्यक्ति है जो विशिष्ट मुद्दों पर काम कर रहा है और अनुरोध खींच रहा है। कभी-कभी समीक्षाकर्ता के रूप में भ्रमित होता है। वास्तव में इसका उपयोग पुल अनुरोध के बजाय मुद्दों के साथ किया जाना है ताकि जब हमें कोई समस्या मिलती है तो हम इसे ठीक करने के लिए किसी को असाइन कर सकते हैं। एक पुल अनुरोध में, एक असाइनर उस व्यक्ति को संदर्भित करता है जो टिप्पणी प्राप्त करने के बाद पुल अनुरोध विलय करने और अन्य रखरखावकर्ताओं से अनुरोध बदलने के प्रभारी होता है।

+2

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

3

स्वीकृत उत्तर के अनुसार। हां, "असाइनर" में एक लूसर परिभाषा है और टीमों की आवश्यकता के अनुरूप अलग-अलग इस्तेमाल किया जा सकता है।

हमारे देवताओं की 8 टीमों में, अधिकांश पीआर में हमारे पास 1 समीक्षक है, जो परिवर्तनों का सुझाव देता है और अंततः पीआर को मंजूरी देता है। समीक्षा चरण के दौरान, "असाइनर" वह व्यक्ति है जिसने पीआर खोला; बाद में अगर पीआर को अन्य डेवलपर द्वारा उठाया जाता है, तो एक नया "असाइनर" जोड़ा जाता है। एक बार पीआर को अनुमोदित और क्यूए या प्रत्यक्ष विलय के लिए तैयार हो जाने पर, एक नया क्यूए "असाइनरी" जोड़ा जाता है। इस तरह "असाइनर" सूची बढ़ती है।

हम "समनुदेशिती" का उपयोग सामूहिक रूप से निम्नलिखित लोगों को नामित करने के:

  1. पुल अनुरोध लेखक
  2. लेखक (आमतौर पर एक ही रूप में 1)
  3. क्यूए व्यक्ति शामिल
  4. व्यक्ति जिम्मेदार पीआर परिवर्तन सुझावों पर काम कर मर्ज करने के लिए (आमतौर पर 2 या 3 के समान)

"असाइनर" का उपयोग करके पीआर को भविष्य में आसानी से ढूंढने में मदद मिलती है ily। मेरी परियोजना में से एक> 3000 पीआरएस है।

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

और अन्य समान प्रश्नों खोज प्रक्रिया को आसान बनाने। पीआर खोज को कम करने के लिए "मील का पत्थर" उपयोग करने में काफी मददगार हैं।

Screenshot Github, Q4 2017

संबंधित मुद्दे