المساهمة.

يجري تطوير DXPR Builder على GitHub في github.com/dxpr/dxpr_builder. وتتبع كل المساهمات سير العمل الموصوف في CONTRIBUTING.md الخاصة بالمستودع.

سير عمل Git

1. يجب ربط كل طلب سحب بقضية GitHub
2. اصطلاح تسمية الفروع: person/target-branch/#issue-description (مثلًا jur/3.x/#42-fix-carousel-autoplay)
3. اكتبوا التزامات ذرية متبعة دليل alphagov لأسلوب Git
4. يجب أن تحمل طلبات السحب وصفًا قصيرًا للحل
5. أبقوا التغييرات ضمن نطاق القضية: لا تنسيق ولا إعادة هيكلة غير مرتبطين

مالك الشيفرة

@jjroelofs هو مالك الشيفرة. ولا يمكن دمج أي طلبات سحب دون مراجعتهم.

معايير البرمجة

• PHP: معايير Drupal للبرمجة
• JavaScript: ‏ES6+ فقط (لا var ولا jQuery في الشيفرة الجديدة). اتبعوا معايير Airbnb لبرمجة JavaScript مع استثناءات المشروع
• CSS: اتبعوا نظام التصميم للخصائص المخصصة وتسمية الرموز
• يجب أن يتضمن تغيير طريقة showed الخاصة بعنصر ملف dxpr_frontend.js المحدَّث. ولّدوه من المخرج الكامل الذي يطبعه المحرّر في وحدة تحكم المتصفح عندما تنقرون زر حفظ حاوية.
• تحتاج المزايا الجديدة تغطية QA Analytics.

شغّلوا npm install مرة بعد الاستنساخ. فهو يثبّت خطاف Husky السابق للالتزام، الذي يفحص ملفاتكم المُجهَّزة؛ انظر الاختبار.

تشغيل اختبارات E2E

تُشغَّل الاختبارات عبر أوامر بشرطة في تعليقات طلبات السحب:

عندما يحتاج تغيير في المنشئ تغييرًا في الاختبار، يحدّث مهندس ضمان الجودة dxpr_maven على فرع، وتشغّلون /e2e maven_ref=<branch> على طلب السحب الخاص بكم. فيُدمَج طلب سحب المنشئ أولًا، ثم طلب dxpr_maven.

قبل طلب المراجعة

1. نفّذوا التغيير
2. اختبروا بعناية
3. نفّذوا جولة تنظيف/إعادة هيكلة
4. أعيدوا الاختبار للتأكد من عدم كسر شيء
5. التزموا وادفعوا بسجل git نظيف
6. تحققوا من نجاح فحوصات المدقق
7. علّقوا /e2e على طلب السحب لتشغيل اختبارات الانحدار
8. اطلبوا المراجعة بعد نجاح كل الفحوصات