search

xpjp / xpfiat-bot

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

embedを使用した対話botの返答の表示 #84

Closed hir0king closed 6 years ago

hir0king commented 6 years ago

やりたいこと

embedとnegaposiジェムを利用し、botの返答から読みとれる感情によって 表示される顔画像が変わるようにしてよりしゃべってる感を出したい。

レビューポイント

既存の対話botの関数を少し変更しました。

hir0king commented 6 years ago

commit名テキトーに一旦つけてたの忘れていましたorz "既存の対話bot関数を一部変更、?ta_plusのcommandを受け付ける部分を追加" こんな感じでつけるつもりでした、すみません

posiとnegaで切り替える顔画像がまだ見つからず一時的なものなので後で差し替えます

何度も申し訳ないですがレビューよろしくお願いします <( )>

Asuforce commented 6 years ago

ref: https://github.com/xpjp/xpfiat-bot/pull/77#issuecomment-354213921 参照のプルリクの作り直し

Asuforce commented 6 years ago

コミットを分割するメリットとして、手戻りが発生した際に該当箇所だけを戻せば正常な状態にできるという点があります。しかし、現状の https://github.com/xpjp/xpfiat-bot/pull/84/commits/2a6cd1bb4b719e9ecdec49ed15cfef0980b404f7 だと全ての変更が戻ってしまい、手直しするのも大変になってしまいます。なので分割していただきたいです。

"既存の対話bot関数を一部変更、?ta_plusのcommandを受け付ける部分を追加"

このメッセージも複数の変更を含んでしまっているので、既存の対話bot関数を一部変更 と ?ta_plusのcommandを受け付ける部分を追加 に分ける。更に一部変更 や 受け付ける部分 という抽象的な表現をできれば具体的にすると誰にでもわかる内容になるでしょう。今回の内容だと Gemfile の編集で更に1コミット分割できそうです。

長々と書きましたが、不特定多数の開発者が参加する OSS の現場ではコードやコミットメッセージで伝え合うことが多くなるのでこのようなスキルが必要になります。ひとつずつ頑張ってみましょう 💪

hir0king commented 6 years ago

レビューありがとうございます 教えていただきありがとうございます 分割してきます！

hir0king commented 6 years ago

コンフリクトを直して最新のmasterに対して変更をしたはずなのですが pushするときにnon-fast-forwardのエラーがでてrejectになってしまいます 調べて出てきたfetchしてマージしたりdiffをとってみたのですが、変更したい部分しか 変更されてないはずなのですが解決しません。 なぜfesture/xp-emotionにpushできないのかわかるかたおりましたら教えていただけると幸です。

hir0king commented 6 years ago

一応変更後がfeature/emotionになってます

hir0king commented 6 years ago

すみません 解決の兆しが見えなかったので閉じます 複雑にしてしまい申し訳ありません branchは少しだけ残しでおき、後で消します