Skip to content
This repository has been archived by the owner on Apr 30, 2019. It is now read-only.

[Feature] Support for enterprise Github repository #223

Merged
merged 2 commits into from
Sep 26, 2015

Conversation

nlunets
Copy link
Contributor

@nlunets nlunets commented Sep 24, 2015

Add an extra parameters in the tsd.json file to support specifying the url to your github host.

Parameter name is githubHost and by default will point to github.com (so no change to current behavior).

If an enterprise github url is specified instead then the url for requests will be adapted to follow the scheme of enterprise github.

Add an extra parameters in the tsd.json file to support specifying the url to your github host.

Parameter name is githubHost and by default will point to github.com (so no change to current behavior).

If an enterprise github url is specified instead then the url for requests will be adapted to follow the scheme of enterprise github.
Add a test, and make sure to support configuration that do not have the parameter
@nlunets
Copy link
Contributor Author

nlunets commented Sep 24, 2015

I'm not entirely sure why the travis CI build is failing on those API calls, the appveyor one does the same and works :/

@Diullei
Copy link
Member

Diullei commented Sep 24, 2015

Very nice!

The tests has been fixed! I'll integrate this PR today!

thanks!

@nlunets
Copy link
Contributor Author

nlunets commented Sep 24, 2015

Well thank you !

Diullei added a commit that referenced this pull request Sep 26, 2015
[Feature] Support for enterprise Github repository
@Diullei Diullei merged commit 15753b1 into DefinitelyTyped:master Sep 26, 2015
@nlunets
Copy link
Contributor Author

nlunets commented Sep 28, 2015

Do you think you could publish a new version on npm with this code ? :)

@Diullei
Copy link
Member

Diullei commented Oct 10, 2015

Hi @nlunets! We'll add more features/fixes before next release.

Thanks for this contribution :)

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