چرا Code Review اهمیت دارد؟
بررسی کد یا Code Review فرایندی است که در آن یک یا چند همکار، تغییرات یک Pull Request را قبل از ادغام با شاخه‌ی اصلی مطالعه و نقد می‌کنند. این فرایند باعث کشف زودهنگام باگ‌ها، یکسان ماندن سبک کدنویسی در تیم و انتقال دانش بین اعضای تیم می‌شود.
در گیت‌هاب، بازبین می‌تواند روی خطوط مشخصی از دیف (diff) کامنت بگذارد، تغییر را تأیید کند (Approve)، درخواست اصلاح بدهد (Request changes) یا فقط نظر بدهد (Comment). نویسنده‌ی PR پس از دریافت بازخورد، commitهای اصلاحی جدید اضافه می‌کند که به‌صورت خودکار به همان PR اضافه می‌شوند.
چند اصل برای بازخورد سازنده
- روی خودِ کد تمرکز کنید، نه روی فردی که آن را نوشته است.
- دلیل پیشنهاد خود را توضیح دهید، نه فقط اینکه «این را عوض کن».
- مسائل مهم (باگ، امنیت، کارایی) را از سلیقه‌های شخصی (نام‌گذاری، فرمت) تفکیک کنید.
- وقتی کدی خوب نوشته شده، آن را هم بگویید؛ بازخورد فقط درباره‌ی ایرادها نیست.
بسیاری از تیم‌ها با تنظیم قوانین محافظتی روی شاخه (Branch Protection Rules) اجباری می‌کنند که هیچ PR بدون حداقل یک Approve قابل merge نباشد. این تنظیمات از بخش Settings مخزن در گیت‌هاب قابل فعال‌سازی است و کیفیت کد ورودی به main را تضمین می‌کند.