Code Review را از سلیقه جدا کن؛ اول رفتار، ریسک و فهمپذیری
reviewها پر از نظرهای ریز سلیقهای میشوند و باگ منطقی یا تغییر رفتار مهم بین کامنتها گم میشود.
قبل از بحث نام متغیر و formatting، بررسی کن تغییر مسئله را حل میکند، حالت خطا را پوشش میدهد و ریسک ناخواسته ندارد.
چطور دقیقتر به موضوع نگاه کنیم؟
- اول هدف PR و رفتار قبل/بعد را بفهم؛ اگر توضیح روشن نیست درخواست context کن.
- مسیرهای ورودی، خطا، permission و data mutation را بررسی کن.
- بعد به طراحی، duplication و خوانایی نگاه کن و در پایان style را به formatter بسپار.
- کامنت blocker، پیشنهاد و سؤال را از هم جدا بنویس تا نویسنده بداند کدام مورد الزام است.
چرا این موضوع مهم است؟
همه بازخوردها وزن یکسان ندارند. ترتیب بررسی باعث میشود انرژی review روی خطاهای پراثرتر صرف شود.
سؤالهایی که معمولاً بعدش پیش میآید
PR چقدر بزرگ باشد؟
هرچه diff کوچکتر و هدفمندتر باشد review عمیقتر میشود؛ تغییر بزرگ را در صورت امکان مرحلهای کن.
Style را کامنت کنیم؟
قواعد قابل خودکارشدن بهتر است به formatter و linter سپرده شود تا review روی منطق بماند.