कोड समीक्षा की प्रक्रिया

गोल

हमारी समीक्षा प्रक्रिया के कई लक्ष्य हैं:

• फ़ंक्शनलिटी और रीडबिलिटी, दोनों में अच्छी क्वालिटी वाला कोड पक्का करें.
• गड़बड़ी का पता लगाएं, क्योंकि गड़बड़ियां होती हैं.
• एक जैसा स्टाइल बनाए रखें, ताकि कोड बेस के किसी भी हिस्से में आसानी से काम किया जा सके.

ब्लॉकी-सैंपल और मुख्य ब्लॉकली कोड में इस्तेमाल किए जाने वाले हर कोड की समीक्षा की जाती है. भले ही, कोड को कम्यूनिटी में योगदान देने वालों ने लिखा हो या ब्लॉकली टीम के सदस्यों ने.

समीक्षक होने के नाते, हमारा मकसद आपके साथ काम करके बदलाव को ज़्यादा से ज़्यादा बेहतर बनाने की कोशिश करना है. हम चाहते हैं कि आप योगदान देने वालों के तौर पर, अपने अनुरोधों की समीक्षा करके उन्हें मर्ज करने के लिए हमसे बातचीत करें.

प्रोसेस

पीआर समीक्षा प्रक्रिया कुछ चरणों से गुज़रती है:

1. असाइनमेंट
2. सुझाव, राय या शिकायत
3. चर्चा
4. बदलाव
5. दोहराव
6. मर्ज करें!

Assignment

जब आपका पुल का अनुरोध आता है, तो Blockly की टीम का कॉल पर मौजूद सदस्य, एक समीक्षक असाइन करता है.

समीक्षकों को विशेषज्ञता के आधार पर चुना जाता है और वे समान रूप से वर्कलोड बांटते हैं.

समीक्षक को असाइन किए जाने में कुछ दिन लग सकते हैं. उसके बाद, समीक्षा करने में कुछ और दिन लग सकते हैं. चिंता न करें, यह सामान्य बात है.

सुझाव/राय दें या शिकायत करें

फ़ीडबैक स्टेज के दौरान, समीक्षक आपके पीआर पर बदलावों के लिए सुझाव देता है. अपने कोड को Google JavaScript स्टाइल गाइड के मुताबिक बनाने के लिए ये आसान चीज़ें हो सकती हैं. या वे कुछ बड़ी चीज़ें हो सकती हैं, जैसे कि आपसे अपने फ़ंक्शन की परिभाषाओं को फिर से व्यवस्थित करने के लिए कहना.

राय देने वाले लोगों को सलाह दी जाती है कि वे अलग-अलग टिप्पणियां करने के बजाय, GitHub की कोड समीक्षाओं का इस्तेमाल करें. ऐसा करने से, आपको कई टिप्पणियों के बजाय एक ही सूचना मिलती है.

चर्चा

बातचीत के दौरान, आपको अपने सुझाव, शिकायत या राय का जवाब देने का मौका मिलता है. हो सकता है कि समीक्षा में की गई कोई एक टिप्पणी समझ में न आई हो: अब आपके पास साफ़ तौर पर जानकारी मांगने का मौका है. या हो सकता है कि आपके समीक्षक ने बदलाव का अनुरोध किया हो, लेकिन आपको लगता है कि उसका असर हो सकता है: अब आपके पास छेड़छाड़ की कोशिश करने का मौका है.

ध्यान दें: यह तरीका काम करे, इसके लिए दोनों पक्षों को साथ मिलकर काम करने की भावना के साथ चर्चा करनी होगी. लक्ष्य "जीत" पाना नहीं है, बल्कि कुछ ऐसा बनाना है जिस पर आप दोनों को गर्व हो.

पुनरीक्षण

संशोधन चरण वह होता है जहां आप अपने पीआर में बदलाव करते हैं. आम तौर पर, ये बदलाव तब होते हैं, जब आपके समीक्षक ने सुझाव, शिकायत या राय के बारे में बताया होता है.

बदलाव पूरे करने के बाद, समीक्षक को टैग दोबारा समीक्षा करने के लिए कहना, बेहतर होगा.

ध्यान दें: चर्चा और संशोधन एक ही समय पर हो सकते हैं.

दोहराव

बदलाव के चरण के बाद, आपके समीक्षक के पास सुझाव देने का एक और मौका होता है और यह प्रोसेस फिर से शुरू हो जाती है.

अक्सर, दूसरी समीक्षा सरल होती है और उसमें विराम चिह्न और कोड स्टाइल जैसे जानकारी पर फ़ोकस किया जाता है. लेकिन कभी-कभी एक दूसरी समीक्षा काफ़ी बड़ी साबित हो सकती है. आपका पहला समीक्षक किसी दूसरे व्यक्ति को भी इसे देखने के लिए कह सकता है, ताकि वह नया नज़रिया पा सके.

मर्ज करें!

मर्ज चरण के दौरान आपको मशहूर बनने का मौका मिलता है. आपने एक बदलाव बनाया है, उस पर चर्चा की है, और उसमें बदलाव किया है और आखिरकार उसे मर्ज कर दिया है! यह एक बहुत बड़ी उपलब्धि है, जिसे बहुत से लोगों ने शुरू नहीं किया, बल्कि पूरा करना चाहते हैं!

Blockly को बेहतर बनाने में, आपकी कड़ी मेहनत के लिए धन्यवाद. और बधाई हो!