इस commit, PR या MR में हुए "बदलाव" के बारे में यह comment छोड़ना आसान होता है कि वह बदलाव ठीक है या नहीं, लेकिन जो हिस्सा बदला जाना चाहिए था पर बदला नहीं गया, उसे पहचानकर comment करना, code review काफ़ी लंबे समय से करने के बावजूद, मुझे अब भी मुश्किल लगता है। अगर codebase मेरे लिए बहुत परिचित न हो, तो यह और भी ज़्यादा कठिन हो जाता है.
कुछ तरीके दिमाग में आते हैं, लेकिन अगर और उपाय हों तो अच्छा लगेगा। आप लोगों ने यह कैसे किया है, यह जानने की जिज्ञासा है।
- implementation करते समय: ऐसे हिस्सों को, जिन्हें बाद में साथ में बदला जाना चाहिए, इस तरह implement करें कि test code उन्हें साथ में cover कर सके।
- PR लिखते समय: PR template में ऐसे उपयुक्त सवाल जोड़ें जिन पर reviewee विचार कर सके। (e.g., क्या इस PR से मिलते-जुलते उद्देश्य वाला कोई पिछला PR है? उस PR से तुलना करने पर क्या इस PR में कहीं कोई छूटा हुआ हिस्सा है?)
- उस codebase में जब भी 'जो किया जाना चाहिए था लेकिन छूट गया' जैसी घटना हो, उसे रिकॉर्ड करें, और हर बार PR template में उस बात को उदाहरण के रूप में update करें।
- PR review करते समय: review guide में यह जोड़ें कि reviewer ऊपर के सवालों की तरह छूटे हुए हिस्सों को खोजने की कोशिश करे।
अभी कोई टिप्पणी नहीं है.