-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add script thank hubot #1
base: master
Are you sure you want to change the base?
Conversation
robot.respond /THANK YOU$/i, (msg) -> | ||
msg.send "You're welcome." | ||
|
||
robot.respond /ありがとう$/i, (msg) -> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
最後に$
を付けずに、/ありがとう/i
にしたほうが「ありがとう!」とかにも対応できて良いと思いました〜。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
私も @upinetree さんと同じコト考えてましたw
/ありがと(?:.*)/
とかにしたら、「ありがと♡」に返事してくれるのが、なんか親しみが出そう…みたいな
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ゆかおさんに書いて頂いた正規表現(ですよね?)の(?:)の部分が調べてもよく分からず、
(.*)にしてしまいました:baby_chick:
(考えてみたら、push する前に聞けば良かったですね:scream:)
(.*)との違いがあれば教えて頂けますとありがたいです:bow::bow:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
idobataでtalktomeさんに(?:)の部分は最初に一致したものだけを扱うためのものと
教えて頂きましたので、ファイルの修正を行いました。
ファイルは応答する単語が増えたら切り出したりすれば良いかなーと思います! |
@upinetree @Yucato コメントありがとうございます!正規表現を確認しつつ修正してみます。 |
そういえば、そもそもこの Hubot は何語をしゃべるのか? とかを決めてなかったですね。 |
そか、今回のスクリプトはどっちも対応できるように書いてたのですよね、すみません忘れてください 🙇 |
Railsの辞書ファイルみたいにできたら良いんですけどね〜。 |
すごいなつかしい PR 見つけた |
これマージしてもいいかな 🎲 |
Hubotへの感謝の気持ちを伝えるスクリプトを追加してみました。
などが心配です。そしてこのプルリクエストがこの後どうなるかもよく分かっていないです。。。。