תרומה לפרויקט WebRTC

הסכם רישיון

אנחנו ב-WebRTC מקבלים תיקונים/בקשות משיכה (pull requests) של תכונות ותיקוני באגים.

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

שליחת דוגמאות

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

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

הוראות

שליחת התיקון הראשון

כדי להעלות את ה-CL הראשון, צריך לבצע כמה הכנות:

• איך בודקים את הקוד ובונים אותו
• ממלאים את הסכם התורם (ראו למעלה)
• אם זו הפעם הראשונה שאתם שולחים קוד, עליכם להוסיף את השם שלכם (או את שם הארגון שלכם, אם הארגון שלכם חתם על הסכם התורם) ואת פרטי הקשר שלכם לקובץ AUTHORS.
• עוברים אל https://webrtc.googlesource.com/new-password ונכנסים באמצעות חשבון האימייל. זה צריך להיות אותו חשבון שמוחזרים על ידי git config user.email
• לאחר מכן, מריצים את הפקודה: git cl creds-check. אם מופיעות שגיאות, אפשר לבקש עזרה ב-discuss-webrtc

לא תצטרכו לחזור על הפעולות שלמעלה. אחרי כל זה, אתם מוכנים להעלות:

העלאת התיקון הראשון

עכשיו, אחרי שהגדרתם את החשבון, תוכלו לבצע את ההעלאה בפועל:

• עשה כך:

  • נניח שאתם נמצאים בהסתעפות הראשית:
    • git checkout -b my-work-branch

  • ביצוע שינויים, פיתוח גרסת build באופן מקומי והרצת בדיקות באופן מקומי

    • git commit -am "Changed x, and it is working"
    • git cl upload

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

    ממלאים את הבאג בצורה נכונה. יש לציין את הקידומת של הכלי למעקב אחר בעיות ואת מספר הבעיה, מופרדים בנקודתיים, למשל webrtc:123 או chromium:12345. אם אין לכם קידומת למעקב אחר בעיות ומספר בעיה, פשוט מוסיפים None.

    שומרים את הקובץ וסוגרים אותו כדי להמשיך בהעלאה לשרת בדיקת הקוד של WebRTC.

    אם הכל יתבצע כמו שצריך, הפקודה תדפיס קישור כמו https://webrtc-review.googlesource.com/c/src/+/53121.

• לוחצים על הקישור הזה ל-CL

• אם לא נכנסתם לחשבון, לוחצים על הלחצן 'כניסה' בפינה השמאלית העליונה ונכנסים באמצעות כתובת האימייל.

• לוחצים על 'התחלת הבדיקה' ומוסיפים בודק. אפשר למצוא בודקים בקובצי OWNERS ברחבי המאגר (יש לבחור את הבודק הקרוב ביותר לשינויים שלכם)

• משיבים למשוב של הבודקים:

  • ביצוע שינויים, פיתוח מקומי והרצת בדיקות מקומיות
    • git commit -am "Fixed X and Y"
    • git cl upload

• אחרי שהבודק מאשר את התיקון, מבקשים ממנו להוסיף אותו לתור ההוספה למאגר

שימו לב: ב-Windows, צריך להריץ את הפקודות שלמעלה במעטפת Git bash כדי ש-gclient ימצא את הקובץ .gitcookies.

Trybots

אם אתם עובדים הרבה עם WebRTC, אתם יכולים להגיש בקשה לזכויות ניסיון. כלומר, תוכלו להריץ את trybots, שמריצים את כל הבדיקות בכל הפלטפורמות. לשם כך, צריך לדווח על באג ב-bugs.webrtc.org ולבקש מצוות EngProd להעניק לכם הרשאות לניסיון.

כדי להריץ את הבדיקה, מעלים את ה-CL כפי שמתואר למעלה ולוחצים על 'הרצת CQ ללא נתונים' או על 'בחירת Trybots' בממשק המשתמש של Gerrit. לשם כך, צריכות להיות לכם הרשאות לניסיון. אחרת, תוכלו לבקש מהבודק להפעיל את הבוטים בשבילכם.