From 766812ae01fc2a3d6d6c2972114c4d19969a0033 Mon Sep 17 00:00:00 2001 From: Erik Thorelli Date: Mon, 10 Aug 2026 16:07:28 -0700 Subject: [PATCH] ci: align approver gate with CODEOWNERS --- .github/workflows/required-approver.yml | 5 +++-- CHANGELOG.md | 1 + 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/.github/workflows/required-approver.yml b/.github/workflows/required-approver.yml index 43baa2a..afdd5b9 100644 --- a/.github/workflows/required-approver.yml +++ b/.github/workflows/required-approver.yml @@ -22,7 +22,7 @@ jobs: name: verify runs-on: ubuntu-latest env: - REQUIRED_APPROVERS: nehal-a2z,juanpflores + REQUIRED_APPROVERS: nehal-a2z,juanpflores,esthor GH_TOKEN: ${{ github.token }} PR_NUMBER: ${{ github.event.pull_request.number }} REPOSITORY: ${{ github.repository }} @@ -85,7 +85,8 @@ jobs: )" if [[ -z "$approved_by" ]]; then - echo "::error::This PR needs an approval from nehal-a2z or juanpflores on the latest head commit." + required_display="${REQUIRED_APPROVERS//,/ or }" + echo "::error::This PR needs an approval from ${required_display} on the latest head commit." exit 1 fi diff --git a/CHANGELOG.md b/CHANGELOG.md index b51f2ba..9b5897f 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -18,6 +18,7 @@ All notable changes to this repository are documented in this file. ### Changed +- Aligned the required-approver check with the repository-wide CODEOWNERS. - Aligned the shared code-review subagent metadata with Gemini CLI's schema. - Removed alternate detailed-output guidance so review agents use `--agent` exclusively.