شكرًا لك على رغبتك في المساهمة في Exercism!
قبل إنشاء Pull Request، يُرجى قراءة دليل كيف تصنع Pull Request رائعًا، الذي يهدف إلى مساعدتك في الجانب غير التقني من Pull Requests لتحصل على تجربة رائعة وتساعد فريق الصيانة لدينا في الوقت نفسه.
يحتوي هذا المستند على بعض الإرشادات الإضافية الخاصة بـ Exercism لأنواع مختلفة من Pull Requests.
Pull Requests الخاصة بالتمارين
قد يكون إنشاء Pull Request لتمرين مفاهيمي أو تمرين تطبيقي أمرًا مُرهبًا نظرًا للقواعد الكثيرة المحيطة بهذين النوعين من التمارين.
ولهذا السبب، قد تستغرق المراجعة من المشرف ساعتين إلى ثلاث ساعات، وقد تُفضي إلى عشرات التعليقات.
لمساعدتك، سيكون هناك مراجع أساسي واحد يعلّق على Pull Request الخاص بك.
لا تثبط عزيمتك بسبب عدد تعليقات المراجعة؛ فمن الطبيعي تمامًا أن يمر التمرين بعدة عمليات إعادة كتابة قبل الوصول إلى شيء ترضى به أنت والمراجع الأساسي.
توصيات
إليك بعض التوصيات التي تساعد في تسريع العملية أكثر.
اقرأ التوثيق
قبل إنشاء Pull Request لتمرين، احرص على قراءة وثائق التمارين المفاهيمية ووثائق التمارين التطبيقية على التوالي.
تساعد قراءة التوثيق مسبقًا بشكل هائل، إذ ستقل تعليقات المراجعة وسيُدمج Pull Request الخاص بك أسرع بكثير.
افحص التمارين الموجودة (إن وُجدت)
إذا كان المسار يحتوي على تمارين موجودة بالفعل، خذ وقتك في دراستها قليلًا لتكتشف كيف تبدو وكيف بُنيت.
Pull Requests الخاصة بالتمارين التطبيقية
فكّر فيما إذا كان التغيير ينتمي فعلًا إلى problems-specifications
بعض محتويات التمرين التطبيقي (مثل مقدمته) تأتي من بياناته الوصفية (المشتركة) كما هي معرّفة في مستودع problems-specifications.
إذا كنت تنوي تحديث محتوى كهذا، ففكّر فيما إذا كان التغيير قد يفيد مسارات أخرى أيضًا.
إذا كان الأمر كذلك، فيُرجى بدلًا من ذلك فتح Pull Request على البيانات الوصفية للتمرين في مستودع problems-specifications.
توصيات عامة
امتنع عن إجراء مراجعات غير مطلوبة
يجري جميع مراجعات Pull Requests مشرف واحد (أو أكثر) من مشرفي المسار، لأنهم مسؤولون عن اعتماد جميع التغييرات في المستودع.
كما أن إجراء المشرفين للمراجعة يساعد في تجنب ملاحظات متعارضة لكاتب Pull Request.
يمكنك بالطبع ترك تعليق لتقديم المساعدة في المراجعة (خاصة مع Pull Requests الأكبر حجمًا) أو لطرح أسئلة إذا لاحظت شيئًا يبدو كخطأ.
