Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

リファクタリング #11

Merged
merged 7 commits into from
Apr 2, 2025
Merged

リファクタリング #11

merged 7 commits into from
Apr 2, 2025

Conversation

micho7
Copy link
Collaborator

@micho7 micho7 commented Mar 29, 2025

リファクタリングを行います。

変更点

  • controllerパッケージ
    • sakuraパッケージをcontrollerパッケージに統合
    • controllerパッケージのcontroller interfaceを廃止
    • controllerパッケージのdecideNextStateOnXXXを関数からメソッドに変更
  • bashパッケージ
    • パッケージ名をcommandに変更
    • shellを介さずにコマンド実行できるように変更
  • CLI
    • グローバルIPアドレスが付与されているインターフェイス名をCLIで受け取るように変更
    • データベースサービスのポート番号(デフォルト3306)をCLIで受け取るように変更
  • processパッケージ
    • mariadbデーモンの強制終了をkillコマンドからsystemctlコマンドを使うように変更
    • それに伴いsystemCtlConnectorにKillServiceメソッドを追加し、processパッケージは廃止
  • その他
    • goのバージョンアップ
    • 利用モジュールのバージョンアップ
    • ドキュメントの更新
    • バイナリファイル名をsakura-controllerからdb-controllerに変更
    • neighborSet構造体をシンプル化
    • パッケージ外に公開不要な変数をexportしないように変更
    • 不要なループを削除
    • typoを修正

@micho7 micho7 self-assigned this Mar 29, 2025
@micho7 micho7 marked this pull request as draft April 1, 2025 03:15
@micho7 micho7 marked this pull request as ready for review April 2, 2025 11:37
@micho7 micho7 merged commit cc4aa70 into main Apr 2, 2025
1 check passed
@micho7 micho7 deleted the refactor branch April 2, 2025 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant