Edge-home-orchestration-go: [MNEDC]アーキテクチャの観点からのリファクタリング

作成日 2020年11月16日  ·  5コメント  ·  ソース: lf-edge/edge-home-orchestration-go

mnedcmgr関数は機能的に類似しているので、discoverymgrの下にあるべきだと思います。
mnedcmgrコードをdiscoverymgrの下に移動するのはどうですか?
以下を変更する必要があります。

  • [x]パッケージ名をmnedcmgrからmnedcなどに変更します。
  • [] Discoverymgrは、StartMNEDCServer()およびStartMNEDCClient()を呼び出す必要があります。

コメントは大歓迎です!

question refactoring

最も参考になるコメント

@ t25kim @MoonkiHong :この変更を確認しました。 Goがサポートしていない周期的なインポートが含まれているようです。 そのため、同じものを変更してテストすることを確認しています。

全てのコメント5件

@ ayush-kr @ suresh-lc(寄稿者の観点から)あなたの意見はどうですか?

mnedcmgr関数は機能的に類似しているので、discoverymgrの下にあるべきだと思います。
mnedcmgrコードをdiscoverymgrの下に移動するのはどうですか?
以下を変更する必要があります。

  • []パッケージ名をmnedcmgrからmnedcなどに変更します。
  • [] Discoverymgrは、StartMNEDCServer()およびStartMNEDCClient()を呼び出す必要があります。

コメントは大歓迎です!

@ t25kim素晴らしい提案をありがとう。 mnedc貢献者に関して、 mnedcmgrdiscoverymgrに移行する可能性を推定します。

@ t25kim @MoonkiHong :この変更を確認しました。 Goがサポートしていない周期的なインポートが含まれているようです。 そのため、同じものを変更してテストすることを確認しています。

@ t25kim @MoonkiHong :この変更を確認しました。 Goがサポートしていない周期的なインポートが含まれているようです。 そのため、同じものを変更してテストすることを確認しています。

@ suresh-lc迅速な分析をありがとうございます。 あなたが提案したように、同じものを変更してテストすることについてのチェックを完了したら、特に_GoLangがサポートしていない循環インポート_について、最良のアプローチについてさらに議論しましょう。 更新をお待ちしております。 ^^

この問題は、最新のマージされたPR#178がChange package name from mnedcmgr to mnedc or something.解決したため、引き続き有効です。

このページは役に立ちましたか?
0 / 5 - 0 評価