search

xpjp / xpfiat-bot

XP JPのディスコード上で動作しているボットです。 XPをJPYに変換したりする機能があります。
MIT License
20 stars 5 forks source link

対話Bot 英語返答機能の作成 #70

Closed Diaboro87 closed 6 years ago

Diaboro87 commented 6 years ago

何を解決するのか

英語でおしゃべりします。(inputには対応しておりません。)

レビューポイント

CIは通した。あとは頼んだ＿|￣|○ il||li

Diaboro87 commented 6 years ago

レビューお願いします

toshikidoi commented 6 years ago

素敵な機能、ありがとうございます！

本PRは以下のPRと内容が重なっている？気がしたのですが、どういう使い分けのイメージでしたか〜!? https://github.com/xpjp/xpfiat-bot/pull/63

Diaboro87 commented 6 years ago

先の #63 はあくまで翻訳機能の追加です。 こちらは対話Botの品質向上になります。

toshikidoi commented 6 years ago

なるほど！ https://github.com/xpjp/xpfiat-bot/pull/63 がマージされたことで、こちらのFilesChangedが小さくなり、意図を理解出来ました〜！ ありがとうございます😄

Diaboro87 commented 6 years ago

念のため、再度通知。レビューお願いします。

toshikidoi commented 6 years ago

レビュー遅れ気味で申し訳ないです。。。 大変お待たせいたしました。

一部リファクタリングして頂いた方が良さそうな箇所(冗長な箇所やtalk2というメソッド名など)がありますが、TODOコメントを残して頂いているので、LGTMです🎉

Diaboro87 commented 6 years ago

talk2とかは近いうちに直します。すみませんｗ 場合によっては対話Bot切り出してもいいかもしれないので、検討します。

toshikidoi commented 6 years ago

talk2とかは近いうちに直します。すみませんｗ

いえいえ、全然OKですw こういうプロジェクトの場合、実装のモメンタムが一番重要だと思うので、どんどん行きましょうw

場合によっては対話Bot切り出してもいいかもしれないので、検討します。

そうですね、どんどん大きくなっていくようだったら、対話BotだけこのBotから切り離すのも有りだと思います😄