أعتقد أن وظيفة mnedcmgr يجب أن تكون ضمن Discoverymgr نظرًا لأنها متشابهة فيما يتعلق بالوظائف.
ماذا عن نقل كود mnedcmgr تحت Discoverymgr؟
يجب تغيير أدناه.
نرحب بأي تعليقات!
@ ayush-kr @ suresh-lc وماذا عن رأيك (من وجهة نظر المساهمين)؟
أعتقد أن وظيفة mnedcmgr يجب أن تكون ضمن Discoverymgr نظرًا لأنها متشابهة فيما يتعلق بالوظائف.
ماذا عن نقل كود mnedcmgr تحت Discoverymgr؟
يجب تغيير أدناه.
- [] تغيير اسم الحزمة من mnedcmgr إلى mnedc أو شيء من هذا القبيل.
- [] يجب أن يستدعي Discoverymgr StartMNEDCServer () و StartMNEDCClient ().
نرحب بأي تعليقات!
@ t25kim شكرا لك على اقتراح عظيم. سنقدر جدواها لترحيل وظائف mnedcmgr
إلى discoverymgr
مع المساهمين في mnedc
.
@ t25kimMoonkiHong: بحثنا هذا التعديل. يبدو أنه ينطوي على استيراد دوري لا يدعمه Go. لذلك نحن نتحقق من تغييره واختباره.
@ t25kimMoonkiHong: بحثنا هذا التعديل. يبدو أنه ينطوي على استيراد دوري لا يدعمه Go. لذلك نحن نتحقق من تغييره واختباره.
@ suresh-lc شكرًا لك على تحليلك السريع. كما اقترحت ، دعنا نناقش أكثر حول أفضل نهج بمجرد الانتهاء من فحص تغيير الشيء نفسه واختبار الأشياء ، لا سيما حول _الاستيراد الدوري الذي لا تدعمه GoLang_. سننتظر التحديثات الخاصة بك. ^ ^
هذه المشكلة لا تزال سارية منذ أن تم حل آخر PR # 178 المدمج الذي تم حله Change package name from mnedcmgr to mnedc or something.
.
التعليق الأكثر فائدة
@ t25kimMoonkiHong: بحثنا هذا التعديل. يبدو أنه ينطوي على استيراد دوري لا يدعمه Go. لذلك نحن نتحقق من تغييره واختباره.