کد ریویو فقط پیدا کردن باگ نیست؛ هماهنگی، خوانایی و یادگیری تیم هم هست. این چک‌لیست را داشته باشید.
- درستی: آیا کد کار خواسته‌شده را انجام می‌دهد؟ حالت‌های مرزی و خطا چطور؟
- خوانایی: نام‌ها و ساختار روشن‌اند؟ نیاز به کامنت توضیحی هست؟
- تست: رفتار جدید تست دارد؟
- امنیت: ورودی کاربر اعتبارسنجی می‌شود؟ رازها لو نرفته‌اند؟
- عملکرد: کوئری یا حلقهٔ پرهزینه اضافه نشده؟
- اندازهٔ تغییر: PR کوچک و متمرکز است؟
نکتهٔ ارتباطی
دربارهٔ کد حرف بزنید، نه آدم. سؤال بپرسید («چرا این‌طور؟»)، دلیل بدهید و بین نکتهٔ ضروری و پیشنهاد سلیقه‌ای فرق بگذارید.
