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 9bfee8c..d25f749 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -24,6 +24,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.