בדיקת שינוי

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

   • ‫0 אין ציון

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

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

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

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