Skip to content
This repository has been archived by the owner on Aug 26, 2020. It is now read-only.

Parse HUBOT_GITHUB_API, setting host, prefix and protocol. #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gotascii
Copy link

This PR adds the ability to set the host, prefix and protocol client options via the ENV variable HUBOT_GITHUB_API. This conforms to the Hubot ecosystem Github client configuration API (e.g. hubot-deploy, github-merge, etc.).

@msabramo
Copy link

msabramo commented Nov 9, 2015

By any chance, does this make it support GitHub Enterprise?

I'll probably test the adapter with and without this PR to find out for myself eventually, but if folks happened to have tried it before and have any tips, that could save some time.

@msabramo
Copy link

msabramo commented Nov 9, 2015

OK, I was just able to successfully use the code in this PR with GitHub Enterprise by doing:

export HUBOT_GITHUB_API=https://api.github.mycompany.com

PORT=8081 HUBOT_GITHUB_EVENT_NOTIFIER_TYPES=push,pull_request,issue_comment,status HUBOT_GITHUB_TOKEN=<some long token> bin/hubot --adapter github-adapter --name GitHubBot

screen shot 2015-11-09 at 6 46 37 am

So 👍 from me!

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.

2 participants