Eu acho que a função mnedcmgr deve estar sob o discoverymgr, pois eles são semelhantes no que diz respeito à funcionalidade.
Que tal mover o código mnedcmgr sob o discoverymgr?
Abaixo deve ser alterado.
Quaisquer comentários são bem-vindos!
@ ayush-kr @ suresh-lc E a sua opinião (da perspectiva dos contribuidores)?
Eu acho que a função mnedcmgr deve estar sob o discoverymgr, pois eles são semelhantes no que diz respeito à funcionalidade.
Que tal mover o código mnedcmgr sob o discoverymgr?
Abaixo deve ser alterado.
- [] Mude o nome do pacote de mnedcmgr para mnedc ou algo assim.
- [] O discoveryymgr deve chamar StartMNEDCServer () e StartMNEDCClient ().
Quaisquer comentários são bem-vindos!
@ t25kim Obrigado por uma ótima sugestão. Vamos estimar sua viabilidade para migrar as funções de mnedcmgr
para discoverymgr
com relação aos contribuintes de mnedc
.
@ t25kim @MoonkiHong : Verificamos essa modificação. Parece envolver importação cíclica que Go não suporta. Portanto, estamos verificando a alteração do mesmo e testando.
@ t25kim @MoonkiHong : Verificamos essa modificação. Parece envolver importação cíclica que Go não suporta. Portanto, estamos verificando a alteração do mesmo e testando.
@ suresh-lc Obrigado por sua pronta análise. Como você sugeriu, deixe-nos discutir mais sobre a melhor abordagem depois de concluir a verificação sobre como alterar o mesmo material e testar, especialmente sobre a _importação cíclica que GoLang não suporta_. Vamos esperar por suas atualizações. ^^
Este problema ainda é válido, pois o último PR # 178 mesclado resolveu Change package name from mnedcmgr to mnedc or something.
.
Comentários muito úteis
@ t25kim @MoonkiHong : Verificamos essa modificação. Parece envolver importação cíclica que Go não suporta. Portanto, estamos verificando a alteração do mesmo e testando.