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

Picking endpoint from the properties file and environment variables #156

Merged
merged 3 commits into from
Mar 15, 2015

Conversation

ashwanthkumar
Copy link
Contributor

Helps seemless switching between public github and enterprise without any code changes.

Helps seemless switching between public github and enterprise without any code changes
@buildhive
Copy link

Kohsuke Kawaguchi » github-api #280 SUCCESS
This pull request looks good
(what's this?)

@@ -70,7 +70,7 @@ public static GitHubBuilder fromCredentials() throws IOException {

}

public static GitHubBuilder fromEnvironment(String loginVariableName, String passwordVariableName, String oauthVariableName) throws IOException {
public static GitHubBuilder fromEnvironment(String loginVariableName, String passwordVariableName, String oauthVariableName, String endpointVariableName) throws IOException {
Copy link
Contributor

Choose a reason for hiding this comment

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

Will broke backward compatibility

@buildhive
Copy link

Kohsuke Kawaguchi » github-api #293 SUCCESS
This pull request looks good
(what's this?)

kohsuke added a commit that referenced this pull request Mar 15, 2015
Picking endpoint from the properties file and environment variables
@kohsuke kohsuke merged commit e146563 into hub4j:master Mar 15, 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