תהליך בדיקת הקוד

שערים

לתהליך הבדיקה שלנו יש כמה מטרות:

  • הקפידו על קוד איכותי, גם בפונקציונליות וגם בנוחות.
  • תופסים באגים כי יש באגים.
  • שומרים על סגנון עקבי כדי שיהיה קל להתחיל לעבוד עם כל חלק ב-codebase.

כל הקוד שנכלל ב-blockly-samples וב-CoreBlockly עובר בדיקה, גם אם הוא נכתב על ידי תורמי תוכן מהקהילה וגם אם הוא נכתב על ידי חברי הצוות שלBlockly.

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

התהליך

תהליך הבדיקה של מחלקת יחסי הציבור כולל כמה שלבים:

  1. העברת תנאים
  2. משוב
  3. דיון
  4. גרסה
  5. חזרה
  6. מיזוג!

מטלה

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

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

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

משוב

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

מומלץ לבודקים להשתמש בבדיקות הקוד של GitHub (במקום להוסיף תגובות ספציפיות) כדי שתקבלו התראה אחת במקום כמה התראות.

קבוצת הדיון

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

גרסה

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

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

חזרה על מילים

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

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

מיזוג!

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

תודה על כל המאמצים שהשקעת כדי לשפר אתBlockly. כל הכבוד!