Closed Hirotaka-Hanai closed 4 months ago
LGTM
指摘事項はありませんでした。よく実装されていると思います。
もともとの課題ですが、 loggerは、コードが増えすぎないうちに実装した方がよいですね。
あと、コード中のドキュメンテーションをもう少し充実させるか、PRに対して、1つ以上の adr を必須にして、実装のコンセプトの解説があるとよいと思いました。今回の実装は、そんなに複雑ではないので大丈夫なんだけど、もう少し複雑になると、なぜこの実装にしたのか?という解説がないと、読むのが辛くなってしまいそうな気がしました。 adr は別のリポジトリでもよいかもしれないけど。
マージお願いします。
チケット
87 #62
実装内容
未実装