כלים, איכות וצוות · מתחילים

ביקורת קוד שמשפרת ולא מעכבת

בשורה אחת: ביקורת טובה קטנה, מהירה, ומתמקדת בנכונות ובתחזוקתיות — לא בהעדפות סגנון שכלי אוטומטי צריך לאכוף.

גודל

בקשת מיזוג של מאתיים שורות תקבל הערות אמיתיות; של אלפיים תקבל "נראה טוב". פצלו. אם אי אפשר לפצל, כתבו בתיאור מסלול קריאה מומלץ.

על מה מעירים

נכונות, מקרי קצה, אבטחה, ביצועים במקומות רגישים, ובהירות למי שיקרא בעוד שנה. סגנון, רווחים וסדר ייבוא — לכלי אוטומטי, לא לאדם.

נסחו הערות כשאלה או כהצעה, וסמנו מה חוסם ומה רק דעה. "חוסם: זה מאפשר גישה למשתמש אחר" ברור יותר מרמז מנומס.

זמן תגובה

ביקורת שממתינה יומיים עוצרת אנשים ומייצרת ענפים ארוכים. קבעו נורמה — למשל, מגיבים תוך חצי יום — והתייחסו אליה כמו לכל התחייבות צוותית.

לעומק

הכניסו לרשימת התיוג של הביקורת פריטים שקשה לאכוף אוטומטית: האם הוספה בדיקה, האם השינוי תואם לאחור, האם התיעוד עודכן, ומה קורה בכישלון. ארבע השאלות האלה תופסות את רוב מה שנשכח.