چطور کد ریویو رو واقعاً سریع کنیم
خلاصهٔ کاملتر
این مقاله رو Sandor Dargo بعد از تاکی که توی Meeting C++ دربارهٔ کد ریویو داده نوشته. یکی از حاضرها بهش گفته بود که بخش «ریویوهایی که روزها طول میکشن» رو کم پوشش داده، و این پست جواب همون نقده. به گفتهٔ نویسنده، ریویوی کند فقط تحویل رو عقب نمیاندازه، بلکه آرومآروم همهچیز اطرافش رو هم هزینهدار میکنه.
نویسنده میگه هزینهای که کسی اندازهاش نمیگیره همینه: نویسندهٔ PR تا برسه فیدبک بیاد، ذهنش رفته سراغ کار بعدی و باید دوباره swap بشه؛ ریویوور هم بعد از سه روز باید توضیح و تغییرات رو از نو بخونه، و پاس دومش همیشه سطحیتره. تیم هم momentum رو از دست میده و یک PR که یک هفته بازه، merge conflict میخوره و کارهای وابسته رو بلاک میکنه.
به نظر نویسنده بزرگترین اهرم — و سختترینش — اینه که اول ریویو، بعد نوشتن. یعنی وقتی میشینی کار کنی، اول صف ریویوت رو خالی کن و تازه بعدش کد جدید بنویس. میگه اکثر تیمها برعکس عمل میکنن و «وقتی فرصت شد» میرن سراغ ریویو، ولی اون فرصت هیچوقت نمیرسه. تأکیدش اینه که این فقط وقتی جواب میده که کل تیم سرش توافق کنه؛ تنهایی نمیشه اعمالش کرد.
برای PRی که گیر میکنه یک playbook روزشمار پیشنهاد میده: روز اول حتماً یک پاس اولیه یا حتی یک ایموجی 👀 که نویسنده بدونه دیده شده؛ روز دوم اگه رفتوبرگشت ادامه داشت، یک تماس ۱۰ دقیقهای — چون ping-pong نوشتاری معمولاً نشونهٔ یک سوءتفاهمه؛ روز سوم بردن موضوع به standup؛ و بعد از اون، به گفتهٔ نویسنده مشکل از خود process هست نه کد، پس باید PR رو بست یا کوچیکترش کرد. جملهبندی خودش اینه که PRی که بیشتر از سه روز زنده میمونه، بهندرت یک مشکل کدیه.
یک الگوی دیگه شکستن چرخهٔ ping-pong زودهنگامه. میگه قانونی که سعی میکنه رعایت کنه اینه: سه بار رفتوبرگشت روی یک PR؟ ریویو رو تموم کن، شروع کن به حرف زدن. یک pairing پونزدهدقیقهای از شیش راند کامنت بهتره، و اگه همون الگو با همون فرد تکرار میشه، این دیگه فرصت coaching هست نه کد ریویو.
در آخر میگه AI معادله رو عوض کرده: تولید PR ارزون شده ولی ریویو نه، پس صف سریعتر پر میشه. به نظرش کد تولیدشده با AI ریویوی متفاوتی میخواد — اگه نویسنده نتونه از یک انتخاب دفاع کنه، کد آماده نیست، حالا فرقی نمیکنه اون انتخاب از مدل اومده یا از یک آموزش نیمهحفظ. همچنین رباتهای ریویو رو جزئی از همون صف میدونه، نه جدا از اون؛ هشدارهای الکیشون باعث alert fatigue میشه و ریویووری که یاد بگیره ۷۰٪ کامنتها رو رد کنه، اون ۳۰٪ مهم رو هم رد میکنه.
نکات کلیدی:
- اول صف ریویوت رو خالی کن، بعد کد جدید بنویس؛ به نظر نویسنده این بزرگترین اهرمه
- PRی که بیشتر از سه روز باز میمونه معمولاً مشکل کد نیست؛ ریشهاش نیازمندی مبهم یا PR بزرگه
- سه راند رفتوبرگشت روی یک PR یعنی وقت تماس و pairing رسیده، نه کامنت بیشتر
- فیدبک تکراری با یک نفر رو به coaching تبدیل کن، نه کامنت
- رباتهای ریویو پرنویز بیشتر ضرر دارن؛ ریویووری که همهچی رو کامنت کنه، کسی نمیخونتش
