בדיקת שינוי

במסמך הזה מפורטות הוראות לבדיקת שינוי. כדי לבצע שינוי תקין, חשוב לזכור את הקריטריונים הבאים:

  • האם השינוי הזה מתאים למטרה המוצהרת של הפרויקט?
  • האם השינוי הזה תקף בארכיטקטורה הקיימת של הפרויקט?
  • האם יש לשינוי הזה פגמים בעיצוב שעלולים לגרום לבעיות בעתיד?
  • האם השינוי הזה תואם לשיטות המומלצות שנקבעו לפרויקט הזה?
  • האם השינוי הזה הוא דרך טובה לבצע את הפונקציה המתוארת?
  • האם השינוי הזה יוצר סיכוני אבטחה או חוסר יציבות או גורם לשיבושים באפליקציות קיימות?

למידע נוסף על בדיקת שינוי, ראו בדיקת שינוי במסמכי התיעוד של Gerrit.

אם יש לכם תפקיד של בודקי של שינויים, תקבלו אימייל עם קישור לשינוי. כדי לבדוק ולבדוק את השינוי, פועלים לפי השלבים הבאים:

  1. לוחצים על הקישור של האימייל לשינוי. יוצג מסך פרטי השינוי של Gerrit. במסך הזה מוצג תיאור של השינוי, מזהה את הגורם שגרם לשינוי ואת כל הבודקים, ורשימה של הקבצים שמשויכים לשינוי.

  2. קוראים את תיאור השינוי כדי להבין את מטרת השינוי.

  3. כדי לבדוק את השינויים בקובץ, לוחצים עליו. הקובץ נפתח במסך השוואה בין שתי חלוניות, בין הקובץ הקיים בצד שמאל לבין הקובץ עם הצעות לשינויים מצד שמאל. כדי לבדוק את הקובץ:

    1. משתמשים בעכבר כדי להדגיש (לחיצה + גרירה) את החלק בקוד שלגביו יש לכם תגובה או שאלה. מופיעה ההודעה 'צריך להקיש על c כדי להגיב'.

    2. מקישים על c. תופיע תיבת טיוטה של תגובה.

    3. מזינים את התגובה או השאלה ולוחצים על שמירה.

  4. לוחצים על המספר הייחודי של השינוי (בצד שמאל למעלה במסך) כדי לחזור למסך Change Info.

  5. מומלץ להמשיך לבדוק כל קובץ שביצעתם בו את השינוי.

  6. כשמסיימים לבדוק את כל הקבצים בשינוי, חוזרים למסך 'שינוי מידע' ולוחצים על תשובה. יופיע מסך ההצבעה לצורך שליחת הדרישות.

  7. בוחרים אחד מהציונים הבאים לשינוי:

    • 1+ נראה לי טוב, אבל גם מישהו אחר צריך לאשר.

    • 0 אין ציון

    • -1 אני רוצה שההודעה לא תישלח כפי שהיא, יש דברים שצריך לתקן.

    • 2- אין לשלוח את השינוי הזה

  8. לוחצים על תשובה כדי לשלוח את התגובות ואת הציון למפתח השינוי.

אם יש בעיות שצריך לפתור, המפתח מתקן אותן ומבקש מכם לבדוק את התיקונים. אם לא, סיימתם לבדוק את השינוי.