hubot-archive / hubot-pager-me

PagerDuty integration for Hubot
https://www.npmjs.com/package/hubot-pager-me
MIT License
73 stars 92 forks source link

Allow case-insensitive matches for "pager me incident NNN" #34

Closed cce closed 9 years ago

cce commented 9 years ago

Hi! We use this Hubot script a lot, but noticed that the "pager me incident NNN" command doesn't work in our environment. After debugging it turned out that it was because of the mixed-case robot name we use and the fact that the regexp pattern for this command didn't end in /i like all the other robot.respond patterns.

This makes the regexp pattern for this command more similar to the others:

~/hubot-pager-me/src/scripts$ git grep robot.respond
pagerduty.coffee:  robot.respond /pager( me)?$/i, (msg) ->
pagerduty.coffee:  robot.respond /pager(?: me)? as (.*)$/i, (msg) ->
pagerduty.coffee:  robot.respond /pager forget me$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? incident (.*)$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? (inc|incidents|sup|problems)$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? (?:trigger|page) ([\w\-]+)$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? (?:trigger|page) ([\w\-]+) (.+)$/i, (msg) ->
pagerduty.coffee:  robot.respond /(?:pager|major)(?: me)? ack(?:nowledge)? (.+)$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? ack(nowledge)?(!)?$/i, (msg) ->
pagerduty.coffee:  robot.respond /(?:pager|major)(?: me)? res(?:olve)?(?:d)? (.+)$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? res(olve)?(d)?(!)?$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? notes (.+)$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? note ([\d\w]+) (.+)$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? schedules( (.+))?$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? (schedule|overrides)( ([\w\-]+))?( ([^ ]+))?$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? (override) ([\w\-]+) ([\w\-:\+]+) - ([\w\-:\+]+)( (.*))?$/i, (msg) ->
pagerduty.coffee:  robot.respond /(pager|major)( me)? (overrides?) ([\w\-]*) (delete) (.*)$/i, (msg) ->
pagerduty.coffee:  robot.respond /pager( me)? (.+) (\d+)$/i, (msg) ->
pagerduty.coffee:  robot.respond /who('s|s| is|se)? (on call|oncall|on-call)( (?:for )?(.+))?/i, (msg) ->
technicalpickles commented 9 years ago

Released in 2.0.4, thanks!