botan-party / hansel

DiscordからMinecraftゲームサーバ(EC2)を操作するDiscordBot
MIT License
0 stars 1 forks source link

パッケージを分割した #34

Open shokkunrf opened 2 years ago

shokkunrf commented 2 years ago

ECS対応の前段階として、パッケージを分けた エラーハンドリングのベストプラクティスがわからなかったため、実装の改善案などあれば取り入れていきたい "このあたり受け入れがたい実装をしているけどECS対応のときに変更できればいいや"みたいなコメントでもOK

usagiga commented 2 years ago

また、 CI が落ちているので、 CI のワーキングディレクトリを src に変えてやってください。:bow: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#example-set-the-default-shell-and-working-directory

shokkunrf commented 2 years ago

レビューありがとうございます コメントやデモコードなど本当に助かりました:pray:

エラーハンドリングについて hanselの規模的にスタックトレースはなくてもいいかなとの判断で errors を利用しました 必要になってから pkg/errors に移行するのでも遅くはないのかなと思います (その頃には標準ライブラリに輸入されてればいいな)