בדיקת שינוי

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

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

מידע נוסף על בדיקת שינוי זמין בקטע בדיקת שינוי במסמכי התיעוד של Gerrit.

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

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

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

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

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

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

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

4. לוחצים על המספר הייחודי של השינוי (בפינה הימנית העליונה של המסך) כדי לחזור למסך 'פרטי השינוי'.

5. ממשיכים לבדוק כל קובץ בשינוי.

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

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

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

   • 0 ללא ציון

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

   • -2 לא צריך לשלוח את השינוי הזה

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

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