Skip to content

feat(preprod): Add GitHub Enterprise support for size analysis status checks#109348

Merged
trevor-e merged 1 commit intomasterfrom
telkins/ghe-support
Feb 25, 2026
Merged

feat(preprod): Add GitHub Enterprise support for size analysis status checks#109348
trevor-e merged 1 commit intomasterfrom
telkins/ghe-support

Conversation

@trevor-e
Copy link
Member

Add GitHub Enterprise support for size analysis status checks.

The status check provider factory (_get_status_check_provider) only matched IntegrationProviderSlug.GITHUB, so GitHub Enterprise repos would silently get no status checks posted even though GitHubEnterpriseApiClient already implements StatusCheckClient via GitHubBaseClient. This adds GITHUB_ENTERPRISE to the provider check.

Includes an end-to-end test that creates a GHE integration, repo, and commit comparison, and verifies the check run is posted to the correct GHE API URL (/api/v3/repos/.../check-runs).

… checks

The status check provider factory only handled GitHub Cloud, causing
status checks to silently not be created for GitHub Enterprise repos.
Add GitHub Enterprise to the provider check since the underlying
GitHubEnterpriseApiClient already implements StatusCheckClient.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Feb 25, 2026
@trevor-e trevor-e marked this pull request as ready for review February 25, 2026 19:55
@trevor-e trevor-e requested a review from a team as a code owner February 25, 2026 19:55
@trevor-e trevor-e merged commit 41470c0 into master Feb 25, 2026
80 checks passed
@trevor-e trevor-e deleted the telkins/ghe-support branch February 25, 2026 20:13
@github-actions github-actions bot locked and limited conversation to collaborators Mar 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants