Skip to content
This repository has been archived by the owner on May 15, 2024. It is now read-only.

Hipchat gateway #947

Open
wants to merge 14 commits into
base: maint/1.x
Choose a base branch
from
Open

Hipchat gateway #947

wants to merge 14 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jul 5, 2016

Adds support for messages to be sent directly to Hipchat rooms.

NOTE: Requires the hipchat room, api_token and username to be configured under 'gateways/hipchat' in the configuration file.


@logger.debug "payload: #{reported_payload}"

client = HipChat::Client.new(api_token, :api_version => 'v2')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could the client be initialized on pikelet setup rather than every time a message is delivered?

@ali-graham
Copy link
Member

Looks OK to me -- you may want to try locking the oj gem to its previous version to fix the test failures, or investigate what's changed that is making those tests fail.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant