Skip to content
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

More explicit campfire errors #45

Merged
merged 2 commits into from
Oct 26, 2011
Merged

More explicit campfire errors #45

merged 2 commits into from
Oct 26, 2011

Conversation

dmathieu
Copy link

This provides more explicit campfire errors for the following cases :

  • If the token, rooms or account environment vars aren't defined
  • If campfire returns a 401 error (the token is invalid)

@tmm1
Copy link
Contributor

tmm1 commented Oct 26, 2011

Looks great, thanks.

tmm1 added a commit that referenced this pull request Oct 26, 2011
More explicit campfire errors
@tmm1 tmm1 merged commit 176dde3 into hubotio:master Oct 26, 2011
glucero pushed a commit to ProCern/hubot that referenced this pull request Aug 20, 2012
More explicit campfire errors
technicalpickles pushed a commit that referenced this pull request Apr 17, 2023
technicalpickles pushed a commit that referenced this pull request Apr 17, 2023
technicalpickles pushed a commit that referenced this pull request Apr 17, 2023
technicalpickles pushed a commit that referenced this pull request Apr 17, 2023
Currently tests are failing on Window because they assume `/` as the
directory separator.

Refs #45
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.

3 participants