Unified बनाम Split Diff

 (matklad.github.io)
3 पॉइंट द्वारा GN⁺ 2023-10-25 | 1 टिप्पणियां | WhatsApp पर शेयर करें
  • कोड रिव्यू में unified diff और split diff के उपयोग के फायदे और नुकसान समझाए गए हैं
  • unified diff और split diff सरल और छोटे बदलावों के लिए उपयुक्त हैं
  • बड़े और जटिल बदलावों के लिए unified diff या split diff आदर्श नहीं हैं
  • लेखक किसी खास समय बिंदु पर पूरे codebase की समीक्षा करना पसंद करते हैं, हाल ही में बदले गए हिस्सों पर ध्यान केंद्रित करते हुए सामान्य समीक्षा भी करते हैं
  • लेखक सुझाव देते हैं कि आदर्श diff view में बाईं ओर कोड की वर्तमान स्थिति दिखाई जाए, और दाईं ओर हल्के रूप से हाइलाइट किए गए बदलावों के साथ वर्तमान दिख रहे codebase का unified diff दिखाया जाए
  • यह बताया गया है कि यह review format मौजूदा tools में अच्छी तरह समर्थित नहीं है, क्योंकि वे वास्तविक code की बजाय diff की समीक्षा पर केंद्रित हैं
  • लेखक इस review style के लिए low-tech workflow का उपयोग करते हैं, और लोकल में pull request जांचने के लिए एक script का उपयोग करते हैं. यह script pull request के सभी commits हटा देती है, लेकिन सभी बदलाव बने रहते हैं
  • लेखक का workflow बदली गई files को आसानी से नेविगेट और review करने तथा reviewed hunks को चिह्नित करने की सुविधा देता है, लेकिन status buffer और editor में वर्तमान में खुली file के बीच automatic sync की कमी है
  • लेखक ऐसा tool चाहते हैं जो इस तरीके से code review करना आसान बनाए, और इसके लिए custom ad-hoc tool बनाने की आवश्यकता न पड़े
  • लेखक यह भी बताते हैं कि हालांकि लेख code review के तरीकों पर चर्चा करता है, code review का मुख्य लक्ष्य जरूरी नहीं कि सिर्फ code की समीक्षा करना ही हो, और इस विषय पर संबंधित पोस्ट्स के लिंक भी साझा करता है

संबंधित पढ़ाई

1 टिप्पणियां

 
GN⁺ 2023-10-25
Hacker News राय
  • यह लेख code review में unified diff और split diff के अंतर पर चर्चा करता है।
  • कुछ commenters का कहना है कि review का प्रकार team और ticket के अनुसार बदलता है; कुछ लोग दूसरी नज़र से होने वाली मानसिक जाँच को पसंद करते हैं, जबकि अन्य गहरे, संरचित, pre-merge feature review को प्राथमिकता देते हैं।
  • difftastic नामक एक tool का उल्लेख है, जो अधिक सूक्ष्म diff highlighting के लिए structural diffing का उपयोग करता है।
  • कुछ commenters PR खोलकर review की जाने वाली changes की जाँच करने के लिए vim के साथ scripting का उपयोग करते हैं।
  • बड़े और जटिल codebase में code review की कठिनाई पर ज़ोर दिया गया है, और समस्या tools से अधिक culture और knowledge sharing से जुड़ी बताई गई है।
  • यह उल्लेख किया गया है कि GitHub की एक सुविधा, browser के भीतर पूरे IDE में जाने के लिए . दबाना, पूरे file के context में changes देखने के लिए उपयोगी है।
  • कुछ commenters split diff में अनावश्यक context हटाने की लेखक की पसंद पर सवाल उठाते हैं, जबकि कुछ अन्य p4merge जैसे दूसरे tools की सुविधाओं को मिस करते हैं।
  • browser में diff view देखने के लिए GitHub के VSCode का उपयोग करने का सुझाव दिया गया है, ताकि पूरी file और अधिक पढ़ने योग्य जटिल diffs देखे जा सकें।
  • Meld को ऐसे use case के लिए अच्छी तरह काम करने वाले tool के रूप में सुझाया गया है।