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

WIP: GitHub Enterprise Support #94

Closed

Conversation

johnbellone
Copy link

This is a work in progress.

I would like to be able to use this inside of my firm with our GitHub Enterprise appliance. In order to do that there needs to be a few changes. This pull-request will track everything necessary to get the project working!

@owenthereal
Copy link
Owner

⚡ Let me know when it's ready for code review

The easiest way to support this initially seems to be adding a host
parameter here. Ideally we should probably read from the Git
configuration somehow. If we're going to be pushing to a corporate
installation *and* GitHub proper this may be problematic.
The configuration from disk should pass the host parameter up through
Octokit and use the proper API URL.
@owenthereal
Copy link
Owner

@johnbellone Do you need any help with this PR?

@johnbellone
Copy link
Author

I just haven't got around to finishing. If you have free time go ahead! I should be able to free up by next Monday.

@owenthereal
Copy link
Owner

I wish I could. I don't have any copy of GHE to test against. Is it that only URL is different?

@johnbellone
Copy link
Author

Yeah, that's pretty much it. I just need to also have support for adding a proxy, etc.

@owenthereal owenthereal mentioned this pull request Dec 2, 2013
@calavera calavera mentioned this pull request Dec 8, 2013
3 tasks
@calavera
Copy link
Contributor

Master works with github enterprise now, and #115 makes it work slightly better. I think this PR can be closed.

@owenthereal
Copy link
Owner

@calavera Hooray! Very happy to see it works! 👯

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.

None yet

3 participants