Closed s9rt closed 6 years ago
masterの内容を含むようrebaseしてコンフリクトの内容にしてみました。このような変更でよろしかったでしょうか?
CI のエラーの対応お願いします〜
@s9rt 面白いアイデアありがとうございます! そして、実装問題なさそうです!
1点。
本PRはdevelop
に向いていますので、実装頂いた内容を本番に反映するためには、本PRをdevelop
にマージ後にdevelop
をmaster
にマージする必要があります。
幸い、本PRの途中でmaster
を取り込んで頂いたので、問題はないと思いますが、忘備録として。。。 🙇
(現状、そもそもdevelop
がmaster
についていけてないですし、私含め皆さんmaster
に直接PR出すという漢の開発フローで進めてますのでw、このPRの向き先もmaster
に変えて頂いても良いかもです)
本番へのデプロイ周りは @p-suke さんや @Asuforce さんにお任せ状態なので、いったん私はApprove
はせずに口頭でのLGTMとさせて頂きます〜🎉
baseをmasterにし、コンフリクトを解決しました。
あ、また master が進んでいるのでコンフリクトだけ直してください〜
コンフリクト修正しました!
何を解決するのか
Botに絵文字を含む文を返させることで少しばかりの遊び心が加わる
レビューポイント
yukichi絵文字が実装される予定なのでそのタイミングでマージしていただければ…