Edge-home-orchestration-go: [MNEDC] إعادة بناء ديون من حيث الهندسة المعمارية

تم إنشاؤها على ١٦ نوفمبر ٢٠٢٠  ·  5تعليقات  ·  مصدر: lf-edge/edge-home-orchestration-go

أعتقد أن وظيفة mnedcmgr يجب أن تكون ضمن Discoverymgr نظرًا لأنها متشابهة فيما يتعلق بالوظائف.
ماذا عن نقل كود mnedcmgr تحت Discoverymgr؟
يجب تغيير أدناه.

  • [x] تغيير اسم الحزمة من mnedcmgr إلى mnedc أو شيء من هذا القبيل.
  • [] يجب أن يستدعي Discoverymgr StartMNEDCServer () و StartMNEDCClient ().

نرحب بأي تعليقات!

question refactoring

التعليق الأكثر فائدة

@ t25kimMoonkiHong: بحثنا هذا التعديل. يبدو أنه ينطوي على استيراد دوري لا يدعمه Go. لذلك نحن نتحقق من تغييره واختباره.

ال 5 كومينتر

@ 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. .

هل كانت هذه الصفحة مفيدة؟
0 / 5 - 0 التقييمات