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 GitHub enterprise #165

Merged
merged 4 commits into from
Mar 14, 2017
Merged

Conversation

mashbourne
Copy link
Contributor

Support for DANGER_GITHUB_API_BASE_URL env var http://danger.systems/guides/getting_started.html#enterprise-github

Resolves #153

Copy link
Member

@macklinu macklinu left a comment

Choose a reason for hiding this comment

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

This change is looking good - thanks for contributing! The build is failing due to lint errors on these two lines though.

@@ -129,8 +129,10 @@ export class GitHubAPI {
if (this.token !== undefined) {
headers["Authorization"] = `token ${this.token}`
}

return this.fetch(`https://api.github.com/${path}`, {

Copy link
Member

Choose a reason for hiding this comment

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

Trailing whitespace

return this.fetch(`https://api.github.com/${path}`, {

const baseUrl = process.env["DANGER_GITHUB_API_BASE_URL"] || "https://api.github.com"

Copy link
Member

Choose a reason for hiding this comment

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

Trailing whitespace

@codecov-io
Copy link

codecov-io commented Mar 14, 2017

Codecov Report

Merging #165 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #165   +/-   ##
=====================================
  Coverage       0%     0%           
=====================================
  Files           3      3           
  Lines          93     93           
  Branches       17     17           
=====================================
  Misses         93     93

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8efdf2b...82a3997. Read the comment docs.

@deecewan
Copy link
Member

Thanks for the changes!

@deecewan deecewan merged commit d31a0fc into danger:master Mar 14, 2017
@alex3165
Copy link
Contributor

When do you plan to release this ?

@orta
Copy link
Member

orta commented Mar 14, 2017

very soon, just wanted to get #156 in and then do a bit of cleanup now

@orta
Copy link
Member

orta commented Mar 14, 2017

shipped in 0.12.0

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

6 participants