סקירה ומיזוג של בקשת משיכה
המדריך הזה מראה לכם איך לסקור ולמזג בקשת משיכה.
פתיחת משטח העבודה של בקשת המשיכה
קטע בשם “פתיחת משטח העבודה של בקשת המשיכה”- עברו אל בקשות משיכה בסרגל הצד
- בחרו בקשת משיכה מהרשימה
בקשת משיכה נפתחת כמשטח עבודה עם לשוניות, ולא כדף בודד. חמש לשוניות נפתחות כברירת מחדל, בסדר הזה: סקירה כללית, הבדלים, בקרת מקור, צ’אט ופעולות (בדיקות CI מקובצות לפי תהליך עבודה). סקירה — ההתקדמות של פייפליין הסקירה, ואחר כך הארטיפקט שהוא פרסם עם מסילת הממצאים שלו — במכוון אינה אחת מהן: לבקשת משיכה אין סקירה עד שמישהו מבקש אחת, ולכן שאל את ה־AI הוא שפותח אותה. טרמינל, עורך לקובץ בודד, code-server מוטמע ודפדפן בתוך האפליקציה נפתחים לפי דרישה; לשוניות של תצוגה מקדימה לפריסה מוזרקות אוטומטית כשמזוהה כתובת URL של תצוגה מקדימה בבקשת המשיכה. פריסת הלשוניות שלכם נשמרת לכל בקשת משיכה בנפרד.
לשונית סקירה כללית נושאת את המטא-נתונים של בקשת המשיכה (מחבר, ענף, תוויות, אחראים, ריאקציות, קומיטים, מצב הבדיקות) ואת שורת הפעולות: רענון, סקירה, מזג, פתיחה ב-IDE ותפריט הצפה ⋯.
קריאת ה-diff
קטע בשם “קריאת ה-diff”פתחו את לשונית הבדלים כדי לראות את כל ה-diff של בקשת המשיכה בבת אחת, עם עץ קבצים למעבר בין קבצים.
- כפתור ההגדרות בסרגל הכלים של ה-diff מחליף בין תצוגה מפוצלת (זה לצד זה) לתצוגה מאוחדת, ומפעיל או מכבה גלישת שורות וליגטורות בגופן הקוד.
- כותרות hunk מסוג
@@מוחלפות בלחצנים Show N lines / Show end of file שמלבישים בלחיצה את ההקשר שמסביב. - הקישו על
j/kכדי לדלג בין קבצים, עלcכדי לקפל או לפרוס את הקובץ שבמיקוד, עלtכדי להחזיר את סרגל הצד לעץ הקבצים ועל⌘Fכדי לחפש בכל ה-diff (Enter/Shift+Enterעוברים בין ההתאמות). - הקישו על
vכדי לסמן את הקובץ שבמיקוד כנצפה, והסמן מתקדם לקובץ הבא. ב-GitHub מצב הצפייה נשמר במארח הקוד, כך שהוא עוקב אחריכם בין לקוחות; ל-GitLab ול-Bitbucket אין API כזה, ולכן שם הוא נזכר רק במכשיר הזה.
גרירת עכבר באזור הקוד בוחרת טקסט עד העמודה המדויקת, ו-⌘C מעתיק את קוד המקור
הגולמי ולא את סמני ה-diff המוצגים.
השארת הערות מוטבעות
קטע בשם “השארת הערות מוטבעות”- הקישו על השוליים של שורת קוד כדי לפתוח תיבת כתיבה המעוגנת לאותה שורה, או
השתמשו בלחצן
+בכותרת קובץ להערה ברמת הקובץ - כתבו את המשוב שלכם ב-Markdown
- שלחו
ב-GitHub וב-GitLab תיבת הכתיבה מציעה שתי דרכי שליחה: הוספת תגובה בודדת מפרסמת מיד, והתחלת סקירה / הוספה לסקירה משייכת את התגובה לסקירה ממתינה שיוצאת עם אישור, בקש שינויים או תגובה. ל-Bitbucket אין קיבוץ לסקירה ממתינה, ולכן שם כל שליחה מפרסמת מיד. פרסום מיידי מציג מצב ממתין עד שמארח הקוד מאשר אותו, ואז מתחלף למסונכרן. אם הפרסום נכשל, השרשור מסומן בשגיאה ומציע ניסיון חוזר.
רק הגוף של הסקירה הוא טיוטה: הוא נשמר בצד השרת לפי (בעלים, מאגר, מספר בקשת המשיכה) רגע אחרי שאתם מפסיקים להקליד, כך שהוא שורד רענון ועוקב אחריכם בין לקוח שולחן העבודה ללקוח הווב.
שליחת סקירה
קטע בשם “שליחת סקירה”- לחצו על סקירה בשורת הפעולות של סקירה כללית (הוא מוסתר בבקשות המשיכה שלכם ובבקשת משיכה סגורה או מוזגת)
- כתבו את גוף הסקירה. אפשר לצרף תמונה, להוסיף אימוג’י או GIF ולהחליף בין כתיבה לתצוגה מקדימה
- לחצו על בקש שינויים, תגובה או אישור
ההכרעה היא הלחצן — אין בורר הכרעה נפרד. אחרי אישור או בקש שינויים, הלחצן מתויג מחדש כאושר או ביקש שינויים, וגוף הטיוטה נמחק. הכרעה של תגובה משאירה את הלחצן בתווית סקירה.
מיזוג
קטע בשם “מיזוג”הלחצן מזג מופיע כשיש לכם הרשאת כתיבה ובקשת המשיכה פתוחה ואינה טיוטה. הצבע שלו הוא אות מוכנות הנגזר מהאישורים הנדרשים וממצב הבדיקות (מוכן / ממתין / חסום) — והוא בגדר המלצה בלבד. מצב הסקירה לעולם אינו מנטרל את הלחצן; הגנת הענפים של מארח הקוד עצמו היא שאכופה את הכללים.
- לחצו על מזג כדי לפתוח את חלונית המיזוג
- בחרו שיטת מיזוג: squash, merge commit או rebase (squash הוא ברירת המחדל). ל-Bitbucket אין מיזוג rebase אמיתי, ולכן rebase ממופה לאסטרטגיית ה-fast-forward שלו ונכשל בענף שהסתעף
- ב-squash וב-merge commit, ערכו את כותרת הקומיט והתיאור שמולאו מראש. rebase לא מקבל אף אחד מהם ומסתיר את שני השדות
- קראו את חלונית האזהרות — היא מפרטת בדיקות שנכשלו וסקירות שממתינות, אבל אינה חוסמת אתכם
- לחצו על לחצן המיזוג בחלונית כדי לאשר
המיזוג אינו הפיך: הוא דוחף קומיטים למארח הקוד, ולכן הוא מקבל שלב אישור במקום
רשומת ביטול. אחר כך המטמון של בקשת המשיכה מבוטל, הסטטוס שלה מתחלף למוזג,
ואירוע PullRequestStatusChanged נורה עם הסטטוס merged. האירוע הזה משחרר את עץ
העבודה הזמני של „פתיחה בעורך” ומפעיל שני פייפליינים מובנים מראש: Stale
repository cleanup, שמסיר את עצי העבודה ותיקיות השיחה הישנים של בקשת המשיכה,
וRelease notes, שמנסח רשומה ביומן השינויים.
העברת בקשת המשיכה לסוכני סקירה
קטע בשם “העברת בקשת המשיכה לסוכני סקירה”תפריט ההצפה ⋯ בלשונית סקירה כללית מכיל את שאל את ה־AI, שמפעיל את
הפייפליין המובנה מראש pr_review עבור בקשת המשיכה הזו. הוא מופיע רק כשבקשת
המשיכה פתוחה. ראו
שימוש בסקירה מבוססת AI כדי לדעת מה
הפייפליין עושה ואיך לפרסם את הממצאים שלו, וחישוב
סקירה למודל הדטרמיניסטי של קבוצות
וצירים.
סקירת בקשת משיכה חיצונית
קטע בשם “סקירת בקשת משיכה חיצונית”Control Center מייבא גם בקשות משיכה שנפתחו בידי אנשים מחוץ לאפליקציה. זרימת הסקירה שלמעלה זהה. בקשות משיכה חיצוניות מזוהות על ידי שירות הסקירה המחזורית של בקשות משיכה.