Edge-home-orchestration-go: [MNEDC] Refactorización en términos de arquitectura

Creado en 16 nov. 2020  ·  5Comentarios  ·  Fuente: lf-edge/edge-home-orchestration-go

Creo que la función mnedcmgr debería estar bajo discoverymgr ya que son similares con respecto a la funcionalidad.
¿Qué tal mover el código mnedcmgr debajo de discoverymgr?
Debe cambiarse a continuación.

  • [x] Cambie el nombre del paquete de mnedcmgr a mnedc o algo.
  • [] El discoverymgr debe llamar a StartMNEDCServer () y StartMNEDCClient ().

¡Cualquier comentario es bienvenido!

question refactoring

Comentario más útil

@ t25kim @MoonkiHong :

Todos 5 comentarios

@ ayush-kr @ suresh-lc ¿Qué opinas de tu opinión (desde la perspectiva de los colaboradores)?

Creo que la función mnedcmgr debería estar bajo discoverymgr ya que son similares con respecto a la funcionalidad.
¿Qué tal mover el código mnedcmgr debajo de discoverymgr?
Debe cambiarse a continuación.

  • [] Cambie el nombre del paquete de mnedcmgr a mnedc o algo.
  • [] El discoverymgr debe llamar a StartMNEDCServer () y StartMNEDCClient ().

¡Cualquier comentario es bienvenido!

@ t25kim Gracias por una gran sugerencia. Estimaremos su viabilidad para migrar las funciones de mnedcmgr a discoverymgr con respecto a los contribuyentes de mnedc .

@ t25kim @MoonkiHong :

@ t25kim @MoonkiHong :

@ suresh-lc Gracias por su rápido análisis. Como sugirió, permítanos discutir más sobre el mejor enfoque una vez que haya completado la verificación para cambiar lo mismo y probar cosas, especialmente sobre la importación cíclica que GoLang no admite. Esperaremos sus actualizaciones. ^^

Este problema sigue siendo válido ya que el último PR # 178 combinado resolvió Change package name from mnedcmgr to mnedc or something. .

¿Fue útil esta página
0 / 5 - 0 calificaciones