Skip to content

ci: split PR checks into explicit job groups#61

Merged
rian-be merged 5 commits into
mainfrom
codex/split-pr-checks
Jul 1, 2026
Merged

ci: split PR checks into explicit job groups#61
rian-be merged 5 commits into
mainfrom
codex/split-pr-checks

Conversation

@rian-be

@rian-be rian-be commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR keeps the PR checks readable by splitting examples out of the main CI workflow and leaving build/test verification in pr-check.yml.

Highlights

CI workflow

  • Keep pr-check.yml focused on build and test validation.
  • Move example validation into a separate examples-check.yml workflow.
  • Preserve explicit jobs for core examples, governance examples, and the Redis example.
  • Keep job names specific so failures point at the affected surface.

Governance tests

  • Add a dedicated approval-request helper for the target-mapping scenario.
  • Align the approval workflow test with the request shape it validates.

Why

The previous workflow was correct but still too tall in the GitHub Actions UI. Splitting example validation into a second workflow keeps the checks easier to scan while preserving explicit coverage for each repository surface.

Checks

  • dotnet build ModularityKit.Mutator.slnx -c Release
  • dotnet test Tests/ModularityKit.Mutator.Governance.Tests/ModularityKit.Mutator.Governance.Tests.csproj -c Release
  • dotnet test Tests/ModularityKit.Mutator.Tests/ModularityKit.Mutator.Tests.csproj -c Release --no-build --no-restore
  • dotnet test Tests/ModularityKit.Mutator.Governance.Redis.Tests/ModularityKit.Mutator.Governance.Redis.Tests.csproj -c Release --no-build --no-restore
  • dotnet run --project Examples/Core/BillingQuotas/BillingQuotas.csproj -c Release --no-build --no-restore
  • dotnet run --project Examples/Governance/RequestLifecycle/RequestLifecycle.csproj -c Release --no-build --no-restore

Closes

@github-actions github-actions Bot added ci CI/CD and repository automation changes tests Test coverage and test changes labels Jul 1, 2026
@rian-be rian-be changed the title [codex] split PR checks into explicit job groups ci: split PR checks into explicit job groups Jul 1, 2026
@rian-be rian-be self-assigned this Jul 1, 2026
@rian-be rian-be force-pushed the codex/split-pr-checks branch from 912fa2f to a98fa84 Compare July 1, 2026 12:57
@rian-be rian-be marked this pull request as ready for review July 1, 2026 13:06
@rian-be rian-be merged commit 2e31b1b into main Jul 1, 2026
24 checks passed
@rian-be rian-be deleted the codex/split-pr-checks branch July 1, 2026 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI/CD and repository automation changes tests Test coverage and test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task]: Add CI matrix for examples and test packages

1 participant