elastic / jarvis

Logstash's ChatOps bot, J.A.R.V.I.S.
41 stars 28 forks source link

Make the publish a merge command be more tolerant on the number of spaces #29

Closed ph closed 8 years ago

ph commented 9 years ago

If you have more space between the publish and the url it wont work and @jarvis will not tell you why (similar behavior with the merge command)

# this command doesn't work
@Jarvis2 publish  https://github.com/logstash-plugins/logstash-input-s3
# this command work
@Jarvis2 publish https://github.com/logstash-plugins/logstash-input-s3
ph commented 8 years ago

pretty sure that clamp solve this :D

jordansissel commented 8 years ago

+1