מתי לעצור ולבדוק — self-review לפני push
עד עכשיו למדנו לבדוק PR שכבר נפתח, אחרי ש-CI כבר רץ. בשיעור הזה חוזרים צעד אחורה, לרגע שלפני שבכלל דוחפים (push): הרגע שבו הסוכן סיים לערוך קוד, ואת/ה עדיין לבד מול ה-diff, לפני שהוא הופך לבעיה של מישהו אחר. self-review הוא ההרגל לקרוא את ה-diff המלא, להריץ טסטים ולינט מקומית, ולשאול: האם
לפני שיוצאים מהבית עם תיק ארוז, פותחים אותו שוב ובודקים בעצמכם מה באמת נמצא בפנים. לא מספיק שהרוכסן סגור ונראה מסודר מבחוץ — צריך לוודא שלא נכנס לשם משהו שלא שמת בכוונה.
- ביקורת עצמית (self-review) לפני push
- הרגל של קריאת ה-diff המלא והרצת בדיקות מקומיות לפני שדוחפים קוד, כדי לתפוס בעיה לפני שהיא הופכת ל-PR ציבורי שמישהו אחר צריך לחשוף.
- git diff --stat
- סקירה מהירה של כל הקבצים ששונו ומספר השורות שהשתנו בכל אחד — הצעד הראשון בסקירה עצמית, לפני שנכנסים לפרטי כל קובץ.
- שינוי-לוואי שקט
- עריכה שמופיעה באותו diff אך אינה קשורה למשימה המבוקשת; היא עוברת CI בשקט כי אף טסט קיים לא בודק אותה, ורק קריאת ה-diff עצמו חושפת אותה.
- בדיקה מקומית לפני push
- הרצת לינט וטסטים על המחשב שלך, לפני שדוחפים, כדי לתפוס תקלות ברגע הכי זול לתקן אותן — לפני שהן הופכות לפידבק ציבורי על PR פתוח.
