تلقي مراجعة طلب السحب

تم إرسال طلب السحب الخاص بك، وقد اجتاز اختبار التكامل المستمر (CI). وهو الآن جاهز للمراجعة.

tl;dr - عملية المراجعة

النسخة المختصرة من عملية المراجعة:

1. انتظر المراجعة.
2. الرد على التعليقات.
3. إذا طُلبت تغييرات:
4. اعمل على إكمال التغييرات المطلوبة.
5. أرسل جميع التغييرات المطلوبة.
6. أعد طلب المراجعة بعد تقديم جميع التغييرات المطلوبة.
7. كرر القسم الثالث حتى لا تكون هناك حاجة إلى مزيد من التغييرات.
8. انتظر حتى تتم الموافقة على طلب السحب ودمجه.

تهانينا! لقد قمت للتو بالتبرع لـ BeeWare!

لقد أرسلت طلب السحب الخاص بي، ما هي الخطوة التالية؟

بعد إرسال طلب السحب، ستحتاج إلى انتظار مراجعة مساهمتك. هناك جانبان لعملية المراجعة: تقديم مراجعة وتلقي مراجعة.

مراجعة التوقعات

يجب أن تتوقع أن يتبع أي شخص يراجع مشاركاتك هذه الإرشادات، بما في ذلك المراجعات من أعضاء الفريق الأساسي. يجب عليك أيضًا اتباع هذه الإرشادات عند مراجعة مشاركات الآخرين.

إذا كنت تشعر أن المراجع الخاص بك يبتعد عن هذه التوقعات، وتشعر بالراحة في إثارة هذه المسألة بنفسك في طلب السحب، فيمكنك القيام بذلك. إذا لم تكن مرتاحًا، فيرجى التواصل مع فريق الاستجابة لقواعد السلوك في BeeWare. سنراجع تقريرك ونتابع الأمر مع المراجع. ستعكس المتابعة الإجراء المبلغ عنه؛ فقد تؤدي المخالفة البسيطة إلى مناقشة، بينما قد تؤدي المخالفة الجسيمة إلى إجراء أكثر خطورة.

تقديم مراجعة

نرحب بجميع المساهمات في مراجعة أي طلب سحب. تصف هذه الإرشادات توقعاتنا من المراجعة، بغض النظر عما إذا كانت مقدمة من أحد أعضاء الفريق الأساسي أو أحد أعضاء المجتمع.

سيحتاج أحد أعضاء الفريق الأساسي دائمًا إلى تقديم المراجعة النهائية؛ ولكن المراجعات من أعضاء المجتمع يمكن أن تكون وسيلة مفيدة لتبسيط العملية - ومن الناحية المثالية، ستكون مراجعة الفريق الأساسي إجراءً شكليًا بعد أن تحدد مراجعات المجتمع جميع المشكلات الرئيسية.

تلقي مراجعة

يتضمن تلقي المراجعة ثلاث خطوات أساسية:

1. التعليقات والأسئلة الأولية.
2. طلبات التغيير.
3. الموافقة والدمج.

كل خطوة مفصلة أدناه. إذا كانت لديك أي أسئلة في أي مرحلة من مراحل العملية، فلا تتردد في طرحها! يسعدنا تقديم المساعدة.

الجدول الزمني والتعليقات الأولية

يهدف الفريق الأساسي إلى ضمان مراجعة كل طلب سحب في غضون عشرة أيام عمل. ومع ذلك، قد يتم تمديد هذا الجدول الزمني في حالة الطلبات الأكثر تعقيدًا، أو عند تقديم طلب سحب أثناء إجازة أعضاء الفريق.

نحافظ عادةً على الاستمرارية مع المراجعين في كل طلب سحب - أي أنك ستعمل على الأرجح مع نفس المراجع طوال عملية المراجعة. وهذا يعني أن المراجع سيكون على دراية بالسياق طوال العملية، وستتمكن من معرفة ما يمكن توقعه من حيث وتيرة الردود وأسلوب المراجعة. إذا وجد المراجع الأولي أن لديه الخبرة اللازمة لمراجعة طلبات السحب الخاصة بك، أو علم أنه لن يكون متاحًا لسبب ما، فقد يمرر مسؤولية طلب السحب الخاص بك إلى عضو آخر في الفريق.

يمكنك أن تتوقع منا الرد على كل تبادل في غضون عشرة أيام عمل متتالية. الرد على التعليقات والأسئلة هو جزء أساسي من عملية المراجعة. سنتوقع ردًا منك قبل الانتقال إلى الخطوة التالية في العملية.

طلبات التغيير

في معظم الأحيان، سيطلب المراجع إجراء تغييرات على طلب السحب الخاص بك. هذا لا يعكس بالضرورة جودة عملك، بل هو مجرد جزء من العملية.

إذا كشفت المراجعة الأولية عن عدد كبير من المشكلات، فقد لا تكون المراجعة الأولى شاملة. وبدلاً من ذلك، ستركز على توفير توجيهات عالية المستوى بشأن العمل المطلوب لجعل طلب السحب في حالة قابلة للدمج. قد تتضمن عملية المراجعة أسئلة لتوضيح الغرض ونطاق العمل الذي تمت محاولته.

العمل على التغييرات المطلوبة

سيقوم المراجع بنشر تعليقات على طلب السحب الخاص بك. يمكن أن تكون هذه التعليقات عامة أو تتعلق بملف معين أو سطر أو سطور معينة من التعليمات البرمجية. قد تتضمن أحيانًا تغييرات مقترحة مباشرة يمكنك تطبيقها على طلب السحب الخاص بك من خلال واجهة مستخدم GitHub. عادةً ما تكون هذه التعليقات عبارة عن أسئلة أو طلبات توضيح أو إرشادات بشأن التحديثات.

وضع علامة على المحادثة على أنها تم حلها

أثناء الجزء الخاص بالمناقشة من عملية التغذية الراجعة، يجب ألا تضع علامة "تم حلها" على أي محادثة بدأها المراجع. وضع علامة "تم حلها" على المحادثة هو مسؤولية المراجع. فالأمر متروك له ليقرر ما إذا كان المشكل المحدد قد تم حله أم لا.

إذا كشفت المراجعة عن وجود مشكلة منهجية (مثل عدم اتساق التسميات في الكود)، فقد لا يسلط المراجع الضوء على كل حالة من حالات هذه المشكلة. بدلاً من ذلك، قد يختار بضعة أمثلة على المشكلة، ويشير إلى أنه ينبغي تصحيح الحالات الأخرى أيضًا. إذا سلطت المراجعة الضوء على مشكلة في مكان واحد، واعتقدت أنها قد تنطبق في أماكن أخرى، فيجب عليك إصلاح هذه المشكلة أينما وجدت. إذا كنت غير متأكد، فاطلب توضيحًا من المراجع.

إرسال جميع التغييرات المطلوبة

بمجرد الانتهاء من جميع التغييرات المطلوبة، يمكنك إرسال تحديث إلى طلب السحب الخاص بك. سيؤدي ذلك إلى تشغيل CI جديد؛ بمجرد التأكد من أن CI لا يزال ناجحًا، انشر تعليقًا تطلب فيه مراجعة محدثة وسيقوم الفريق الأساسي بإعادة النظر في طلب السحب الخاص بك.

ادفع، لا تجبر أو تعيد التأسيس

عند تحديث طلب السحب أثناء المراجعة، من المهم ترك سجل الالتزامات كما هو. لا يهم إذا كانت هناك قائمة ضخمة من الالتزامات؛ فجميعها يتم ضغطها عند دمج طلب السحب. إذا قمت بالدفع القسري أو إعادة تأسيس طلب السحب في منتصف المراجعة، فقد تقوم بإزالة سياق مهم يحتاجه المراجع.

إعادة طلب المراجعة

بمجرد الانتهاء من إجراء جميع التغييرات المطلوبة في مراجعة معينة، ومرور CI مرة أخرى، يمكنك إعادة طلب مراجعة من المراجع. إذا كانت المشكلة معقدة بشكل خاص، وكان إصلاحها سيؤثر على شيء آخر، يمكنك طلب مراجعة الجزء المحدد الذي قمت بتحديثه. سيتم افتراض أن أي طلب للمراجعة هو طلب لمراجعة كاملة. إذا لم تكن مستعدًا لمراجعة كاملة، فتأكد من تحديد ما تريده بالضبط.

الموافقة على طلب السحب والدمج

بمجرد الرد على جميع طلبات التغيير وإتمامها، سيتم الموافقة على طلب السحب. في معظم الحالات، بمجرد الموافقة على طلب السحب، سنقوم بدمجه على الفور. في بعض الحالات، قد تكون هناك ظروف مخففة، مثل الاعتماد على طلب سحب آخر لم يتم دمجه بعد، مما يؤدي إلى تأخير. سنقوم بالإبلاغ عن ذلك في التعليقات، حتى تكون على علم بالوضع.