I think I had to log in to find the URL to use.

Oops. Sorry about the comma. From memory I used similar text already
existing in the script.

I am traveling at the moment so I will not be able to have another look at
this for at least a month.

On 3/06/2013, at 5:45 PM, Josh Nichols <notifications@github.com> wrote:

In src/scripts/time.coffee:

> @@ -27,7 +29,10 @@ module.exports = (robot) ->
>      unless process.env.HUBOT_WWO_API_KEY
>        msg.send 'Please, set HUBOT_WWO_API_KEY environment variable'
>        return
> -    msg.http('http://www.worldweatheronline.com/feed/tz.ashx')
> +    unless process.env.HUBOT_WWO_API_URL
> +      msg.send 'Please, set HUBOT_WWO_API_URL environment variable'

Is there a URL you can go to to figure out what the URL would be? Might be
worth mentioning that in the response.

Also: grammatically, there doesn't need to be a comma after please.

—
Reply to this email directly or view it on
GitHub<https://github.com/github/hubot-scripts/pull/963/files#r4502454>
.