כל קובץ pull request מֵאֶת Coroid עובר תהליך אימות לפני שאתם רואים אותו. פרק זה מסביר מה המשמעות של כך וכיצד להגדיר זאת.
השכבות
תהליך האימות אינו אחיד. ישנן ארבע שכבות, כל אחת עונה על שאלה שונה.
סט הבדיקות שלכם. זהו האות החזק ביותר הזמין, מכיוון שהוא כבר משקף את מה שצוותכם קבע כחשוב. Coroid מריץ אותו בסביבת עבודה נקייה עבור כל משימה.
שערי איכות. בדיקות המוגדרות על פי מדיניות – כיסוי קוד, linting, סריקת אבטחה, בדיקות דפדפן, וה בדיקת גלובליזציה עבור פרויקטים רב-לשוניים. שערים חוסמים עוצרים את המשימה; שערים מייעצים רושמים ממצא ומאפשרים המשך. ראו מדיניות איכות ושערי איכות.
סקירה. סוכן הסקירה קורא את ה־diff המושלם ומפיק ממצאים. ממצאים שניתן לתקן יוחזרו למפתח כעבודה נוספת; השאר יצורפו ל־pull request.
ראיות. פריסות תצוגה מקדימה, תוצאות סריקה ושינויים בכיסוי הקוד מצורפים ישירות ל־pull request, כך שהאימות מגיע יחד עם הקוד. ראו ראיות לסקירה.
- חבילת הבדיקות שלךהאם השינוי פוגע במה שכבר עובד?
- שערי איכותהאם הוא עומד בתקנים שמגדירה המדיניות שלכם?
- סקירההאם קורא קפדני של ה־diff יקבל אותו?
- ראיותהאם הראיות מצורפות לצד הקוד?
- הסקירה שלךהאם זה השינוי הנכון לבצע?
כל שכבה אוטומטית פועלת לפני ש־pull request נוצר, כך שכישלונות חוזרים כעבודה מחדש במסגרת הריצה.
מה שתהליך זה מוכיח ומה לא
תהליך האימות מאשר שהשינוי עושה את מה שהוא אמור לעשות, שאינו שובר את מה שהיה קיים, ושהוא עומד בתקנים שקבעתם.
הוא אכן אינו מאשר שהשינוי הוא הדבר הנכון לפתח. זו המטרה של קריטריוני הקבלה ב־ מפרט ושל תהליך הסקירה שלכם.
שינוי יכול לעבור את כל השכבות ועדיין להיות שגוי. שמירה על ההבחנה הזו חדה היא מה שהופך את שאר התהליך לאמין – ברגע שהאימות מוצג כשיפוט, אנשים מפסיקים לבצע סקירה.
המטרה של הרצתו מראש
כל שכבה פועלת לפני ש־pull request נוצר, כך שכישלונות חוזרים כעבודה נוספת בתוך אותו ריצוי ולא כ־pull request שתידחה.
זה זול יותר לכשל בסביבת העבודה מאשר על המסך שלכם. הסקירה שלכם צריכה להיות ה־ אנושית בדיקה ראשונה, לא המבט הראשון.
מאיפה להתחיל
אם אתם מגדירים זאת בפעם הראשונה:
- ודאו שסט הבדיקות שלכם פועל – ראו הגדרת בנייה ובדיקה. שום דבר אחר אינו רלוונטי אם זה אינו תקין.
- הגדירו מדיניות ברירת מחדל לארגון ברמת ה־ סטנדרט ולא ברמת הקפדנית ביותר שלכם.
- חברו את ספקי הראיות שאתם כבר משתמשים בהם.
- הוסיפו שערים אחד אחד, תחילה אלו המייעצים.