क्या टीएफएस में लेखक को अस्वीकार या प्रतीक्षा करनी चाहिए जब चीजें तय होनी हैं?


13

TFS में, जब हम किसी चीज़ के लिए टिप्पणियों को एक पुल्क अनुरोध में तय करने से पहले रखते हैं, तो हम उसे स्वीकार करेंगे, तो क्या हमें इसे अस्वीकार या लेखक की प्रतीक्षा करनी चाहिए? कौनसा अच्छा है?

यहां छवि विवरण दर्ज करें

जवाबों:


15

पुल कोड के साथ समीक्षा कोड पर Microsoft के अनुसार : अनुमोदन के प्रत्येक वर्ग के सुझाए गए परिवर्तनों को वोट करें :

सुझावों के साथ स्वीकृति दें: पुल अनुरोध के साथ सहमत हों, लेकिन कोड को बेहतर बनाने के लिए वैकल्पिक सुझाव प्रदान करें।

लेखक की प्रतीक्षा : परिवर्तनों को स्वीकार न करें, और लेखक से अपनी टिप्पणियों की समीक्षा करने के लिए कहें। लेखक को आपको यह बताने देना चाहिए कि जब आपने अपनी चिंताओं को संबोधित किया है तो आपको कोड की फिर से समीक्षा करनी चाहिए।

अस्वीकृत : परिवर्तन स्वीकार्य नहीं हैं। यदि आप इस तरह से मतदान कर रहे हैं, तो आपको पुल अनुरोध में एक टिप्पणी छोड़ देनी चाहिए जिसमें परिवर्तन को अस्वीकार कर दिया गया था।

इस प्रकार मेरा Waiting for Authorमतलब यह है कि आपको लगता है कि लेखक ने समाधान के लिए उसके दृष्टिकोण को खराब कर दिया है, लेकिन यदि वह आपकी टिप्पणियों को दिल से लेता है तो उसका कोड रिडीम है।

और Rejectedइसका मतलब है कि नरक में कोई रास्ता नहीं है कि आप इस तरह से कोई बदलाव स्वीकार कर रहे हैं, चाहे कोई भी कोड कितना अच्छा लिखा हो।

आपको अपने आप से जो सवाल पूछना है, वह यह है कि क्या आपके समूह के सुझाए गए उद्देश्य का विचार Microsoft के विचार से मेल खाता है।


1
इसके बारे में: "लेखक को आपको यह बताना चाहिए कि जब आपको अपनी चिंताओं को संबोधित करने के बाद आपको कोड की पुन: समीक्षा करनी चाहिए।" - क्या लेखक को समीक्षकों को सचेत करने के लिए एक तंत्र है कि पीआर पुन: समीक्षा के लिए तैयार है?
रॉबर्ट सिम

@RobertSim - हम पुनरावलोकन के लिए एक संदेश भेजने के लिए समीक्षकों के तहत 'पिंग' विकल्प का उपयोग करते हैं।
किलार्कोवुक

इसे देखने का दूसरा तरीका: Waiting for Author= मुझे यह सुविधा चाहिए, लेकिन आपने इसे गलत कोडित कर दिया है, और जब तक आप इसे ठीक नहीं कर लेते, मैं इसे स्वीकार नहीं करूंगा। और Rejected= मुझे यह सुविधा नहीं चाहिए कि आप मुझसे समीक्षा करने के लिए कह रहे हैं।
गोकू_डा_मास्टर
हमारी साइट का प्रयोग करके, आप स्वीकार करते हैं कि आपने हमारी Cookie Policy और निजता नीति को पढ़ और समझा लिया है।
Licensed under cc by-sa 3.0 with attribution required.