Skip to content

Conversation

@taniuchi
Copy link

@taniuchi taniuchi commented Apr 9, 2013

ローカルに作業用とリクエスト用のブランチを作成し、リクエスト用のブランチをpushする形を取りました。coffeやhotbotと言った初めての言葉が多かったですが、軽く調べて適当に改良してみました。ただ、動作検証はしてません・・・。hearをrespondに変更しただけなので大丈夫だと思いますが・・・。

@moqada
Copy link
Contributor

moqada commented Apr 9, 2013

機能追加、非常に意欲的でありがたいです!とてもありがたい!ありがとうございます!

…ただ残念なことに、これだと二重投稿しちゃうんですよね…。

調べていただいたっぽいんで知ってるかもしれませんが、

  • hear: 投稿すべてに反応
  • respond: 呼び掛けにのみ反応

なんですよ。
なので、今回のように全く同じ正規表現で hearrespond を書いてしまうと

sqbot こんちわ

とか投稿したときに、「respond: 呼び掛け+正規表現に合致」、「hear: 正規表現に合致」 の両方が反応してしまうんです。

…というわけで、respond のほうを hear とかぶらない正規表現に書きなおすか、泣く泣く respond を消滅させるか…どちらかを実施して、再度プルリクしていただけますかー。

本来「動くコード」は求めていないですし、課題的には完全クリアなんですが、ここまでやっていただいたんで…最後までいきましょう!

あと、細かすぎるところで、あんま重要でもないし自分もよくやるんですが、コミットメッセージtypoっちゃってますね。
全体的に非常におしいですー。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants