بازبینی کد بیش از هر جلسه‌ی رترو، شخصیت یک تیم را نشان می‌دهد. بدش انجام بدهی نمایش است؛ خوبش انجام بدهی مهارت همه را جمع می‌کند.

سوگند بازبین

  1. مهربان، دقیق و کاربردی باش. «این گیج‌کننده است» نویز است؛ «اسم x را requestId بگذار چون دوبار در مسیر خطا استفاده شده» سیگنال است.
  2. تحسین را عمومی کن. «استفاده‌ات از الگوی cursor خوب بود — به من هم یاد داد» هیچ هزینه‌ای ندارد.
  3. سلیقه را از درستی جدا کن. اول مشکلات بلاک‌کننده را بگو؛ استایل پیشنهاد است.

فقط PR کوچک

بازبینی با اندازه‌ی دیف пропорциональ است. بالای ۴۰۰ خط، هیچ‌کس واقعاً نمی‌خواند — فقط نگاه می‌کند. بشکنش.

text
PR ایده‌آل:
- یک دغدغه
- یک بازبین
- کمتر از ۳۰۰ خط
- تیتری مثل پیام کامیت

پیشوند nit

text
nit: این تابع کمکی را جدا کن

یک کلمه‌ی زمینه که می‌گوید «برای مرج شدن مسدود نکن».

دریافت بازخورد

  • نظرها را درباره‌ی کد ببین، نه ارزش تو.
  • به‌جای دفاع کردن بپرس «به‌جایش چه می‌کردی؟»
  • تشکر کن؛ این عادت را تقویت می‌کند.

کارهای تکراری را اتومات کن

لینت، فرمت و تایپ‌چک در CI، تا انسان‌ها فقط درباره‌ی چیزهای مهم حرف بزنند.

بازبینی کد ارزان‌ترین آموزشی است که تیم‌ات می‌تواند بخرد. این آیین را حفظ کن.