تغيير طريقة المشاركة الى ملف JSON

[Mohd-PH]

السلام عليكم

هنا ملف nodejs يقرأ ملف devs.json بمشاريع المطورين ويجهز ملف ال README.md بإستخدام template.md

الآن اللي يحتاجه المطور انه يضيف بياناته الى ملف devs.json بعدها احد ادارة المشروع يقوم بتشغيل ملف OSS-update-readme.js لتوليد ملف README.md بسهولة

كذلك قمت بتحديث قسم طريقة المشاركة نهاية الملف لتوضيح الفكرة

[Alaboudi1]

الله يعطيك العافية، مريت سريع على الكود، هل تقدر تخليه يشتغل في GitHub action. على كل pull request يشتغل os-update-readme.js ؟

[Mohd-PH]

الله يعافيك، خفت اسويه على github action ويدخل في loop ماله نهاية 😆 بحيث نعدل ملف, يشتغل الاكشن، يعدل الملف، يشتغل الاكشن، يعدل الملف، يشتغل الاكشن، يعدل الملف .. الخ

بشوفله طريقة ان شاء الله بحيث اني اضيف environment variable اذا شافه ما يعدل، قلت اسويها حبة حبة

[Alaboudi1]

المفروض يشتغل على كل pull request, شوف هنا https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request

[Alaboudi1]

اتوقع هذا بالضبط الي نحتاجة https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#running-your-workflow-when-a-pull-request-merges

[Mohd-PH]

جربته على repo خاص عندي وضبط كذا، ان شاء الله يضبط على المشروع 😸

[Alaboudi1]

الله يعطيك العافية، بإذن الله راح اراجع الإضافة خلال كم يوم. الموضوع يحتاج شوي فصفصة لانه راح يغير كيف نضيف ونعرض المشاريع

[Mohd-PH]

الله يعافيك يارب

خصوصا معه نقدر نغير طريقة العرض من جدول الى قائمة متى ما بغينا، كذلك نقدر نضيف على ملف الjson بيانات اكثر عن المشروع، مثلا اللغة او المنصة اللي يشتغل عليها المشروع 😋

[Alaboudi1]

الهدف باذن الله شيء مماثل للمجتمع الايراني https://matnbaz.net/en, وش رايك ؟

[Mohd-PH]

جميل جدا!!

لكن نحتاج بيانات اكثر واكبر

[Alaboudi1]

اسف على التاخير، الشي الوحيد الي لازم نغيره وهو طريقة تعديل ملف readme. حاليا التعديل عن طريق http put. المفروض عن طريق تغيير محتوى pull request قبل merge.

[Mohd-PH]

نقدر نسوي كذا؟ ما لقيت طريقة

[Alaboudi1]

ممكن هنا يكون الحل https://docs.github.com/en/actions/using-workflows/using-github-cli-in-workflows

[Mohd-PH]

مدري بس احسن نشوف الاكشن مسوي commit بالتعديل بعد ما نسوي merge ل pull request

وهل اصلا نقدر نغير محتوى ب repo شخص اخر قبل ما نسوي merge ؟

لأن اللي فهمته منك، شخص يرفع pull request من cloned project بحسابه، حنا نشوفه ونوافق او نرفض، اذا وافقنا لازم السكربت يعدل على المحتوى بحسابه قبل ما يسوي merge

اعتقد صلاحيات السكربت تمنع نعدل عليه، وش رايك؟

[Alaboudi1]

صح، اعتقد مانقدر نسوي commit مباشرة على pull request. حاليا انا مانع اي commit مباشرة لل main بس ممكن اسمح فقط لبعض الاشخاص او لل action نسوي commit مباشرة بعد كل pull request.

الخيار الثاني زي ماقلت، بعد كل merge نشوف اذا commit حصل على json، نسوي generation جديد للreadme.

وش رايك افضل، commit مباشرة لكل merge, ولا pull request بعد كل تعديل لل json ؟

[Mohd-PH]

اعتقد commit مباشرة لكل merge اسهل من ناحية اتمتة الموضوع

يبيلي ادور طريقة يشوف اذا كان ملف الjson تعدل او لا، اذا ما تعدل ما يحتاج يسوي شي

[Alaboudi1]

@Mohd-PH عشان ما يتعطلع الشغل خلنا ندمج الكود هذا وبعدين نبني فوقة