Pull Requests וביקורת קוד
בשיעור הקודם למדנו לחבר את המאגר המקומי לשרת (GitHub) ולסנכרן איתו: clone, push, pull. עכשיו נשאל שאלה אחרת לגמרי — לא 'איך מעלים קוד' אלא 'איך צוות נותן לקוד להיכנס בבטחה'. התשובה היא Pull Request: בקשה למזג ענף אחד לתוך אחר (בדרך כלל ענף פיצ'ר לתוך main), שפותחת מקום ייעודי לדיון, לביקורת קוד (cod
Pull Request הוא כמו לומר 'סיימתי חדר במלון — בבקשה תבדקו אותו לפני שמכניסים אורחים'. במקום לדחוף את העבודה שלך ישר לקו הראשי (main), את/ה פותח/ת בקשה: 'הנה השינוי שלי, בואו נדבר עליו, תעירו, נריץ בדיקות — ורק כשהכול תקין, נכניס פנימה'.
- Pull Request (PR)
- בקשה ב-GitHub למזג ענף אחד לתוך אחר (בדרך כלל ענף פיצ'ר לתוך main). היא פותחת מקום לדיון, לביקורת קוד ולבדיקות אוטומטיות לפני שהשינוי נכנס.
- ביקורת קוד (code review)
- כשחבר/ת צוות קורא/ת את השינוי שלך, מעיר/ה הערות ומציע/ה שיפורים בתוך ה-PR — כדי לתפוס באגים, לחלוק ידע ולשמור על איכות.
- אישור (approve)
- הסימון של המבקר/ת ש'נראה לי טוב' — אור ירוק שאומר שמבחינתו/ה אפשר למזג את ה-PR.
- בדיקות אוטומטיות (CI)
- תוכנה שרצה לבד בכל PR ובודקת את הקוד (למשל מריצה את הטסטים). אם בדיקה נכשלת, מתקנים לפני שממזגים.
- מיזוג PR (merge)
- הכפתור שמכניס סוף-סוף את הענף שלך לתוך main אחרי שהביקורת אושרה והבדיקות עברו. אז נהוג למחוק את ענף הפיצ'ר.
