feat: add CI - Required Checks aggregation job#5
Conversation
Add a ci-required-checks job that aggregates all CI job results into a single "CI - Required Checks" status check using the shared devantler-tech/actions/require-checks-in-pr composite action. This allows branch protection to require a single check instead of listing each individual job. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
![botantler[bot]](https://avatars.githubusercontent.com/in/1025824?s=60&v=4){kind=small}
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds a single aggregated GitHub Actions job/status check (“CI - Required Checks”) so branch protection can require one check instead of enumerating all CI jobs.
Changes:
- Add
ci-required-checksjob that depends on existing CI jobs and runs even when upstream jobs fail. - Invoke
devantler-tech/actions/require-checks-in-prto compute a single pass/fail status from upstream job results.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot update to target the latest semver release of devantler-tech/actions |
Agent-Logs-Url: https://github.com/devantler-tech/copilot-plugins/sessions/afca5567-a763-45ec-ad82-130b877a8f6a Co-authored-by: devantler <26203420+devantler@users.noreply.github.com>
Head branch was pushed to by a user without write access
Updated to |
|
@copilot apply changes based on the comments in this thread |
Agent-Logs-Url: https://github.com/devantler-tech/copilot-plugins/sessions/48e39470-5672-4e88-874e-8f8c16d31408 Co-authored-by: devantler <26203420+devantler@users.noreply.github.com>
Applied the review feedback in
|
Description
Adds a
ci-required-checksjob named "CI - Required Checks" that aggregates all CI job results into a single status check using the shareddevantler-tech/actions/require-checks-in-prcomposite action.This allows branch protection to require a single check (
CI - Required Checks) instead of listing each individual job.Changes
.github/workflows/ci.yaml: addedci-required-checksjob that:needsall existing jobs (validate-manifests,discover-skills,validate-spec)if: always()to execute regardless of upstream resultsdevantler-tech/actions/require-checks-in-prto fail if any job failed or was cancelledDependencies
require-checks-in-prcomposite action actions#113 being merged and released. The SHA reference will need to be updated once the action is released.require-checks-in-praction ksail#4209 (migrates ksail to the same shared action)