Skip to content

Copilot autoapprover toevoegen#148

Open
Sam Haeck (shaeck) wants to merge 1 commit into
mainfrom
copilot-autoapprover
Open

Copilot autoapprover toevoegen#148
Sam Haeck (shaeck) wants to merge 1 commit into
mainfrom
copilot-autoapprover

Conversation

@shaeck

@shaeck Sam Haeck (shaeck) commented Jul 9, 2026

Copy link
Copy Markdown

Neemt de copilot-autoapprover-opzet uit AFASSoftware/focus over in deze repo.

closes #149

Wat het doet

Wanneer Copilot als automatische reviewer een PR beoordeelt en in de samenvatting **review nodig: nee** zet, keurt de workflow de PR automatisch goed via de AFASACTION GitHub App.

Bestanden

  • .github/workflows/copilot-autoapprove.yml — de workflow. Reageert alleen op reviews van een bot-account waarvan de login met copilot begint. Guards: slaat door Copilot aangemaakte PR's over, en de auteur moet zelf schrijfrechten hebben (anti prompt-injection — belangrijk omdat deze repo publiek is). De limiet op aantal gewijzigde bestanden is bewust weggelaten.
  • .github/instructions/codereview.instructions.md — review-richtlijnen die Copilot op github.com leest. De parseerbare markers (review nodig, severity-labels, PR-labels) zijn identiek aan focus; de beschikbare PR-labels zijn licht aangepast voor deze content-repo.
  • .github/CODEOWNERS — alle wijzigingen (*) naar @AFASSoftware/development-experience.

Nog nodig om end-to-end te werken

  1. Repo-instelling (handmatig): Settings → Copilot → Code review → op Automatic zetten.
  2. AFASACTION-app: moet op OASContent geïnstalleerd zijn met pull-request write-rechten; de workflow gebruikt vars.AFASACTION_APPID + secrets.AFASACTION_PKEY.
  3. Teamtoegang: @AFASSoftware/development-experience moet write-toegang tot deze repo hebben om als code owner te tellen.

🤖 Generated with Claude Code

Neemt de opzet uit AFASSoftware/focus over: een workflow keurt een PR
automatisch goed wanneer de Copilot-review "review nodig: nee" aangeeft.

- .github/workflows/copilot-autoapprove.yml — auto-approve via AFASACTION app,
  met anti prompt-injection guard (auteur moet schrijfrechten hebben). Geen
  limiet op aantal gewijzigde bestanden.
- .github/instructions/codereview.instructions.md — review-richtlijnen met de
  parseerbare markers (review nodig, severity- en PR-labels).
- .github/CODEOWNERS — alle wijzigingen naar @AFASSoftware/development-experience.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Deze PR introduceert een GitHub Actions-workflow die PR’s automatisch kan goedkeuren wanneer GitHub Copilot in de review-samenvatting **review nodig: nee** vermeldt, inclusief bijbehorende Copilot review-instructies en repo-brede CODEOWNERS-instelling.

Changes:

  • Nieuwe workflow toegevoegd die op Copilot reviews luistert en (onder guards) auto-approve + labels kan toepassen via een GitHub App token.
  • Nieuwe Copilot code review-instructies toegevoegd met vaste markers (review nodig, labels:) en label-lijst.
  • CODEOWNERS ingesteld zodat alle wijzigingen door @AFASSoftware/development-experience gereviewd worden.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
.github/workflows/copilot-autoapprove.yml Voegt auto-approve workflow toe op basis van Copilot review-body markers en past labels toe.
.github/instructions/codereview.instructions.md Definieert Copilot review-format/markers en toegestane labels voor consistente parsing.
.github/CODEOWNERS Stelt development-experience in als code owner voor alle paden.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/copilot-autoapprove.yml
Comment thread .github/workflows/copilot-autoapprove.yml
Comment thread .github/workflows/copilot-autoapprove.yml
Comment thread .github/workflows/copilot-autoapprove.yml
Comment thread .github/instructions/codereview.instructions.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

autoapproves

2 participants