Edge-home-orchestration-go: [MNEDC] Refactoring en terme d'Architecture

Créé le 16 nov. 2020  ·  5Commentaires  ·  Source: lf-edge/edge-home-orchestration-go

Je pense que la fonction mnedcmgr devrait être sous le discoverymgr car elles sont similaires en ce qui concerne les fonctionnalités.
Que diriez-vous de déplacer le code mnedcmgr sous le discoverymgr ?
Ci-dessous doit être modifié.

  • [x] Changez le nom du package de mnedcmgr en mnedc ou quelque chose.
  • [ ] Le discoverymgr doit appeler StartMNEDCServer() et StartMNEDCClient().

Tous les commentaires sont les bienvenus !

question refactoring

Commentaire le plus utile

@t25kim @MoonkiHong : Nous avons vérifié cette modification. Semble impliquer une importation cyclique que Go ne prend pas en charge. Nous vérifions donc de changer le même et testons.

Tous les 5 commentaires

@ayush-kr @suresh-lc Et votre avis (du point de vue des contributeurs) ?

Je pense que la fonction mnedcmgr devrait être sous le discoverymgr car elles sont similaires en ce qui concerne les fonctionnalités.
Que diriez-vous de déplacer le code mnedcmgr sous le discoverymgr ?
Ci-dessous doit être modifié.

  • [ ] Changez le nom du package de mnedcmgr en mnedc ou quelque chose.
  • [ ] Le discoverymgr doit appeler StartMNEDCServer() et StartMNEDCClient().

Tous les commentaires sont les bienvenus !

@t25kim Merci pour cette excellente suggestion. Nous évaluerons sa faisabilité pour migrer les fonctions du mnedcmgr vers discoverymgr avec des contributeurs en ce qui concerne les mnedc .

@t25kim @MoonkiHong : Nous avons vérifié cette modification. Semble impliquer une importation cyclique que Go ne prend pas en charge. Nous vérifions donc de changer le même et testons.

@t25kim @MoonkiHong : Nous avons vérifié cette modification. Semble impliquer une importation cyclique que Go ne prend pas en charge. Nous vérifions donc de changer le même et testons.

@suresh-lc Merci pour votre analyse rapide. Comme vous l'avez suggéré, discutons plus en détail de la meilleure approche une fois que vous avez terminé la vérification de la modification et des tests, en particulier de l'_importation cyclique que GoLang ne prend pas en charge_. Nous attendrons vos mises à jour. ^^

Ce problème est toujours valable puisque le dernier PR n°178 fusionné a résolu Change package name from mnedcmgr to mnedc or something. .

Cette page vous a été utile?
0 / 5 - 0 notes