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

Support enterprise/non-standard Github URLs #77

Closed
wants to merge 1 commit into from
Closed

Support enterprise/non-standard Github URLs #77

wants to merge 1 commit into from

Conversation

nickcampbell18
Copy link

My company uses Github Enterprise, which is found on a URL like so: https://github.mycompany.com.

This PR supports the passing of custom Github endpoints via environment variables (the variables mirror the names used in the Octokit library):

This commit also modifies the git remotes parser to look within custom hostnames instead of public github remotes.

Allow the passing of custom Github endpoints
via environment variables:

* GITHUB_WEB_ENDPOINT (default: https://github.com)
* GITHUB_API_ENDPOINT (default: https://api.github.com)

This commit also modifies the Git remotes parser to
look within custom hostnames instead of public github
remotes.
@nickcampbell18
Copy link
Author

This PR is as much about starting a discussion. This scratched our immediate itch; I'd love to see it in the main gem, but I think at this point the code may be asking for a separate Configuration class.

@geniou
Copy link

geniou commented Apr 2, 2015

👍 I like this - we have the same problem.

@mmozuras
Copy link
Member

@nickcampbell18 sorry for taking a long while to answer. The need for Configuration (.pronto.yml) is clear. I'll try to tackle it for the next release. It shouldn't be very hard, but in case I'll fail, keep this PR and we'll stick with using ENV for now 😄

@Knack
Copy link

Knack commented May 25, 2015

+1

Knack added a commit to Knack/pronto that referenced this pull request May 25, 2015
This change allows to use GHE. Refer to the original PR for more info
@mmozuras
Copy link
Member

@nickcampbell18 I've added ability to configure Pronto via .pronto.yml (unreleased) and at the same time, solved this: #104

Thanks for this pull request, would not have happened without it! 🙇

@mmozuras mmozuras closed this Oct 25, 2015
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.

4 participants