Code review עם AI
בשיעור הקודם למדנו לכתוב תיאור PR שמסכם כמה קומיטים. עכשיו, אחרי שה-PR נפתח וה-CI רץ, מגיע השלב שהכי קל לדלג עליו: לקרוא את ה-diff עצמו, בדיוק כמו שהיינו קוראים PR של מפתח/ת אנושי/ת. CI ירוק אומר שהקוד לא שבר טסט קיים — הוא לא אומר שה-diff עושה בדיוק את מה שביקשנו ולא יותר, ולא אומר ששום שינוי-לוו
לבדוק diff שסוכן כתב זה כמו לבדוק חבילה שהובילו לך הביתה לפני שחותמים על הקבלה — לא מספיק שהאריזה סגורה ונראית תקינה מבחוץ, צריך לפתוח ולוודא שבפנים באמת מה שהזמנת ולא משהו נוסף.
- diff של Pull Request
- התצוגה המלאה של כל השורות שנוספו והוסרו ב-PR, בפורמט unified diff; זה מה שקוראים שורה-שורה כדי לבדוק שהשינוי עושה בדיוק את מה שביקשו.
- בקשת שינויים (Request changes)
- פסיקת ביקורת שחוסמת מיזוג עד שהמחבר/ת יתקן/תתקן נקודות ספציפיות; שונה מהערה חופשית — היא בפועל מונעת merge עד שהנקודה טופלה.
- CI ירוק (ומה הוא לא אומר)
- אישור שהקוד עבר lint, בדיקת טיפוסים וטסטים קיימים; הוא לא מאשר שה-diff מוגבל למה שהטיקט ביקש, ולא בודק ערכי-גבול שאף טסט לא כיסה.
- שינוי לא-קשור בתוך ה-diff
- עריכה שמופיעה באותו PR אך אינה קשורה לתיקון המבוקש; גם אם היא עוברת CI בשקט, יש להוציא אותה או לסקור אותה בנפרד.
