Edge-home-orchestration-go: [MNEDC] 架构方面的重构

创建于 2020-11-16  ·  5评论  ·  资料来源: lf-edge/edge-home-orchestration-go

我认为 mnedcmgr 函数应该在 discoverymgr 之下,因为它们在功能方面是相似的。
把 mnedcmgr 代码移到 discoverymgr 下怎么样?
下面应该改了。

  • [x] 把包名从 mnedcmgr 改成 mnedc 什么的。
  • [ ] 发现管理器应该调用 StartMNEDCServer() 和 StartMNEDCClient()。

欢迎任何意见!

question refactoring

最有用的评论

@t25kim @MoonkiHong :我们检查了这个修改。 似乎涉及 Go 不支持的循环导入。 因此,我们正在检查更改并进行测试。

所有5条评论

@ayush-kr @suresh-lc 您的意见如何(从贡献者的角度来看)?

我认为 mnedcmgr 函数应该在 discoverymgr 之下,因为它们在功能方面是相似的。
把 mnedcmgr 代码移到 discoverymgr 下怎么样?
下面应该改了。

  • [ ] 将包名从 mnedcmgr 更改为 mnedc 之类的。
  • [ ] 发现管理器应该调用 StartMNEDCServer() 和 StartMNEDCClient()。

欢迎任何意见!

@t25kim谢谢你的好建议。 我们将估计其将mnedcmgr的功能迁移到discoverymgr可行性,以及关于mnedc贡献者。

@t25kim @MoonkiHong :我们检查了这个修改。 似乎涉及 Go 不支持的循环导入。 因此,我们正在检查更改并进行测试。

@t25kim @MoonkiHong :我们检查了这个修改。 似乎涉及 Go 不支持的循环导入。 因此,我们正在检查更改并进行测试。

@suresh-lc 感谢您的及时分析。 正如您所建议的,在您完成更改相同和测试内容的检查后,让我们进一步讨论最佳方法,尤其是关于 _GoLang 不支持的循环导入_。 我们将等待您的更新。 ^^

这个问题仍然有效,因为最新合并的 PR #178 解决了Change package name from mnedcmgr to mnedc or something.

此页面是否有帮助?
0 / 5 - 0 等级