Skip to content

Add ghe support#3720

Open
arhowe00 wants to merge 1 commit into
Kitware:masterfrom
arhowe00:add-ghe-support
Open

Add ghe support#3720
arhowe00 wants to merge 1 commit into
Kitware:masterfrom
arhowe00:add-ghe-support

Conversation

@arhowe00
Copy link
Copy Markdown

This change makes it easier to configure GitHub integration for users who may use a private GHE instance. CDash's GitHub integration (check runs, commit statuses, PR comments) is currently hardcoded to api.github.com.

I also added a commit that removes some dead code and brings in a utility

Enable CDash to post check runs and commit statuses to ghe installations
in addition to github.com.

Changes:
- Add GITHUB_ENTERPRISE_URL config option (env var)
- Pass enterprise URL to GitHubClient constructor with apiVersion=null
  (so it defaults to 'v3') so the PathPrepend plugin builds correct
  /api/v3/ paths for ghe
- Add isGitHubUrl() helpers in GitHub.php and RepositoryUtils.php to
  match repository URLs against both github.com and the configured GHE
  host
- Convert get_github_api_url() to construct /api/v3/repos/ paths for GHE
  repositories
- Make DoneHandler always create/update the check when a revision exists,
  rather than requiring the pendingSubmissions recheck flag
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants