Skip to content

docs: Document reporting.checks for GitHub Check Runs#1129

Merged
gjkim42 merged 1 commit into
mainfrom
kelos-task-1123
May 13, 2026
Merged

docs: Document reporting.checks for GitHub Check Runs#1129
gjkim42 merged 1 commit into
mainfrom
kelos-task-1123

Conversation

@kelos-bot
Copy link
Copy Markdown

@kelos-bot kelos-bot Bot commented May 12, 2026

What type of PR is this?

/kind docs

What this PR does / why we need it:

The reporting.checks sub-field of GitHubReporting is fully implemented but undocumented in docs/reference.md and docs/integration.md. New users wanting to make a Kelos agent run a required status check for branch protection have no way to discover the feature.

This PR adds:

  • docs/reference.md: three new rows

    • spec.when.githubPullRequests.reporting.checks.name
    • spec.when.githubWebhook.reporting.enabled
    • spec.when.githubWebhook.reporting.checks.name

    Each row notes the checks:write permission requirement, the default name (Kelos: <taskspawner-name>), and the relevant CEL constraints (rejected on githubIssues; webhook source requires at least one of pull_request, pull_request_review, pull_request_review_comment, pull_request_target).

  • docs/integration.md: a status reporting block in the GitHub Pull Requests section with a YAML example, branch protection walkthrough, and a cross-reference to the webhook source. A short status reporting paragraph is also added to the GitHub Webhooks section.

Which issue(s) this PR is related to:

Fixes #1123

Special notes for your reviewer:

Docs-only change; no API or behavior modifications. make verify passes.

Does this PR introduce a user-facing change?

NONE

Summary by cubic

Documented reporting.checks for GitHub PRs and webhooks so Kelos tasks can create Check Runs used by branch protection and merge queues. Adds reference rows and an integration example (YAML), and notes the checks:write permission and CEL validation rules; fixes #1123.

Written for commit 7f7bb9b. Summary will update on new commits.

Add reference.md rows for spec.when.githubPullRequests.reporting.checks.name
and spec.when.githubWebhook.reporting.{enabled,checks.name}, noting the
checks:write permission requirement and the CEL constraints (rejected on
githubIssues; requires a PR event type on githubWebhook).

Extend integration.md with a branch protection / merge queue walkthrough
in the GitHub Pull Requests section and a status reporting note in the
GitHub Webhooks section.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

kelos-bot[bot] has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@github-actions github-actions Bot added needs-actor release-note-none kind/docs and removed needs-kind Indicates an issue or PR lacks a kind/* label labels May 12, 2026
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@gjkim42 gjkim42 added this pull request to the merge queue May 13, 2026
Merged via the queue into main with commit f8b8f2c May 13, 2026
37 of 38 checks passed
@gjkim42 gjkim42 deleted the kelos-task-1123 branch May 13, 2026 02:10
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.

Docs: reporting.checks (GitHub Check Runs for branch protection) undocumented — reference.md only shows reporting.enabled

1 participant