GitHub पर, समीक्षक और असाइनमेंट में क्या अंतर है?


185

GitHub ब्लॉग पर घोषणा की गई 7 दिसंबर, 2016 को एक सुविधा जोड़ी गई, जिसमें समीक्षकों को एक पुल अनुरोध में जोड़ने का विकल्प पेश किया गया।

GitHub समीक्षा विकल्प

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

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

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

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


1
पहली बार "असाइन करने की सुविधा" कब जारी होगी? क्या कोई लेख है जो इसे पेश करता है?
बाबेह

जवाबों:


132

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

कई ओएसएस अनुरक्षकों के साथ चर्चा करने के बाद, समीक्षकों को परिभाषित किया जाता है कि शब्द को क्या माना जाता है: समीक्षा करने के लिए (किसी का कोड) और "असाइन" में नीचे वर्णित एक शिथिलता निश्चित है।

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

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

पिछला उत्तर:

ठीक है मैं आगे जाऊंगा और अपने प्रश्न का उत्तर दूंगा।

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

लेखन-अभिगम (योगदानकर्ताओं के बाहर) के उपयोगकर्ताओं के लिए: लेखन-अभिगम वाला कोई व्यक्ति स्वयं (या अन्य लेखन-निजी सदस्य) को पीआर (समीक्षक) की समीक्षा करने के लिए निर्दिष्ट करेगा। अस्संगिनी रिक्त है।

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


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

क्या एक निर्धारिती के पास लिखने की पहुँच होनी चाहिए?
एम्रे सुल्लुन

क्या दोनों के बीच ईमेल सूचना व्यवहार में कोई अंतर है?
jxramos

26

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

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


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

14

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

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

हम "असाइन" का उपयोग सामूहिक रूप से निम्नलिखित लोगों को नामित करने के लिए करते हैं:

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

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

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

या केवल author:raya-dumasसभी आइटम लेखक (मुद्दों, पीआर) को खोजने के लिए

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

स्क्रीनशॉट Github, Q4 2017


बहुत अच्छी तरह से समझाया गया।
नितिन गौड़

यह उल्लेख किया जाना चाहिए कि आप लेखक के लिए बस खोज कर सकते हैं: मेरे-जीथुब-हैंडल को खोजने के लिए कि पीआर के व्यक्ति ने क्या बनाया
विसैनकस

1

GitHub से पहले केवल एक कार्यक्षेत्र और कोई समीक्षक फ़ील्ड नहीं था। तब कोई भेद नहीं था, इसलिए समीक्षक क्षेत्र का उपयोग आमतौर पर समीक्षक क्षेत्र के रूप में किया जाता था।

लेकिन जो भी आपके प्रोजेक्ट के अनुकूल हो, उनका उपयोग करें।

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