मेरी कंपनी की कई टीमों ने एक कोड समीक्षा वर्कफ़्लो का अभ्यास किया है जो मैंने पहले कभी नहीं देखा। मैं इसके पीछे की सोच को समझने की कोशिश कर रहा हूं, इस विचार के साथ कि पूरी कंपनी को सुसंगत बनाने में मूल्य है। (मैं कई कोडबेस में योगदान देता हूं और अतीत में मतभेदों के कारण फंस गया हूं।)
- कोड लेखक एक पुल अनुरोध प्रस्तुत करता है
- समीक्षक कोड की जांच करता है
- यदि समीक्षक अनुमोदन करता है, तो वे "अच्छा लग रहा है, मर्ज करने के लिए स्वतंत्र महसूस" की तर्ज पर एक टिप्पणी छोड़ते हैं
- यदि समीक्षक को चिंता है, तो वे एक टिप्पणी छोड़ते हैं जैसे "कृपया मामूली मुद्दों को ठीक करें एक्स और वाई, फिर मर्ज करें" (प्रमुख परिवर्तनों के लिए, चरण 2 पर लौटें)
- यदि आवश्यक हो तो कोड लेखक परिवर्तन करता है, और फिर अपने स्वयं के पुल अनुरोध को मर्ज करता है
मेरी निम्नलिखित चिंताएँ हैं:
चरण 3 में अनुमोदन के मामले में, यह वर्कफ़्लो पुल अनुरोध लेखक को एक प्रतीत होता है-अनावश्यक राउंडट्रिप बनाता है। समीक्षक, जो पहले से ही कोड को देख रहा है, बस इसे तुरंत मर्ज कर सकता है।
चरण 3 में किए जा रहे परिवर्तनों के मामले में, पुल अनुरोध को मर्ज करने के लिए एजेंसी अब पीआर के लेखक के साथ पूरी तरह से आराम करती है। लेखक के अलावा कोई भी विलय से पहले के बदलावों को नहीं देखेगा।
इस वर्कफ़्लो के कुछ अन्य फायदे या नुकसान क्या हैं? क्या यह वर्कफ़्लो अन्य इंजीनियरिंग टीमों पर आम है?