בדיקת שינוי

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

   • ‫0 ללא ציון

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

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

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

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