מתן ביקורת

שכן הכותב הוא בעל הקוד, ואתה אורח בביתו. האורח המצווה — לא יוזמן בפעם הבאה. ודרשו חכמים: "consider" ו-"maybe" — לשון חכמים היא.

שכן אם לא הובנת — ייתכן שלא הסברת. ומעשה היה ב-reviewer שכתב "כבר אמרתי!" — וכותב הקוד השיב: "לא הבנתי אז, ולא הבנתי עכשיו."

ומעשה היה ב-reviewer שאישר PR של חמש מאות שורות תוך שלוש דקות. ויתמה הכותב, ויאמר: "אפשר לקרוא כל כך מהר?" ויאמר ה-reviewer: "LGTM." ולא הוסיף.

אמרו חכמים: שיחה פנים-אל-פנים (או בזום) פותרת ב-10 דקות מה שthread של 40 הודעות לא יפתור בשלושה ימים.

חתימתך על PR היא עדות. ועד שחתם ולא קרא — עדותו פסולה, ואחריותו שלמה. קרא לפני שתחתום, ואם לא הספקת — אמור שלא הספקת.