בדיקת שינוי

קל לארגן דפים בעזרת אוספים אפשר לשמור ולסווג תוכן על סמך ההעדפות שלך.

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

   • 0 ללא ציון

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

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

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

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