ci: configure copilot review to reduce noise by efiacor · Pull Request #1064 · kptdev/porch

efiacor · 2026-06-19T09:30:49Z

Description

What changed: Added .github/copilot-code-review.yml and .github/workflows/copilot-review.yaml
Why it's needed: Copilot auto-reviews on every push are creating excessive overhead for reviewers
How it works:
- Config file scopes Copilot to skip generated/vendored code and linter-overlap issues, focus on real bugs
- Workflow requests Copilot review only on PR open (not on every synchronize)
- Manual re-request still available via GitHub UI

Related Issue(s)

Supersedes Check if CoPilot has reviewed a PR so that we can turn off automatic CoPilot reviews #1061

Type of Change

Enhancement

Checklist

Testing Instructions

Disable automatic Copilot code review in repo settings (Settings → Copilot → Code review)
Open a new PR — Copilot should be requested as reviewer once
Push additional commits — Copilot should NOT be re-requested
Verify Copilot comments respect the path exclusions and instruction constraints

Additional Notes

Requires a repo admin to disable automatic Copilot reviews in settings for the workflow trigger to be the sole mechanism
Manual /review or reviewer request still works for on-demand reviews after initial open

AI Disclosure

I have used AI in the creation of this PR.

If so, please describe how:

Kiro to draft the copilot-code-review.yml instructions and copilot-review.yaml workflow
The author has fully verified all code

- Add .github/copilot-code-review.yml with project-specific instructions: skip linter-overlap issues, generated code, vendored code, and style nits. Focus reviews on logic errors, concurrency, security, and K8s API correctness. - Add workflow to request Copilot review only on PR open (not on every push). Manual re-request still available via GitHub UI. - Requires disabling automatic Copilot reviews in repo settings. Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>

netlify · 2026-06-19T09:30:59Z

✅ Deploy Preview for kpt-porch ready!

Name	Link
🔨 Latest commit	`c27cd7b`
🔍 Latest deploy log	https://app.netlify.com/projects/kpt-porch/deploys/6a397932d8ca2e000820eda7
😎 Deploy Preview	https://deploy-preview-1064--kpt-porch.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
🤖 Make changes	Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot

Pull request overview

Adds a repo-scoped Copilot Code Review configuration and a GitHub Actions workflow to reduce review noise by (1) constraining what Copilot comments on and (2) requesting Copilot review only when a PR is first opened (instead of on every push/update).

Changes:

Add .github/copilot-code-review.yml to scope Copilot review focus areas and exclude generated/vendored paths.
Add .github/workflows/copilot-review.yaml to request Copilot as a reviewer on pull_request opened.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
.github/workflows/copilot-review.yaml	Adds a workflow that requests Copilot review when a PR is opened.
.github/copilot-code-review.yml	Adds Copilot review instructions and path-based exclusions to reduce noise.

Copilot

Pull request overview

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

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>

liamfallon

Hopefully it'll listen to you.

liamfallon · 2026-06-22T08:08:11Z

In conjunction with this, I'd be more comfortable if we required all conversations be resolved as well.

Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>

Copilot

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>

Copilot

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>

Copilot

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

sonarqubecloud · 2026-06-22T18:11:37Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

CsatariGergely · 2026-06-23T06:04:13Z

In theory Copilot picks up the instructions from AGENTS.md. I would prefer to have all agent related instructions in one place.
I'm already working on a PR about AGENTS.md for kpt here: kptdev/kpt#4573
What if we merge this PR and once the kpt AGENTS.md is in I will create a pr for AGENTS.md to Porch and merge the contents of .github/copilot-code-review.yml there?

Ye, I was going off this guide.
Main objective is to reduce the noise on PRs by restricting the number of reviews that are run on the PR.

efiacor requested review from a team and Copilot June 19, 2026 09:30

efiacor added the enhancement New feature or request label Jun 19, 2026

efiacor self-assigned this Jun 19, 2026

efiacor requested review from kispaljr, liamfallon and mozesl-nokia as code owners June 19, 2026 09:30

Copilot started reviewing on behalf of efiacor June 19, 2026 09:31 View session

dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 19, 2026

Copilot AI reviewed Jun 19, 2026

View reviewed changes

Comment thread .github/workflows/copilot-review.yaml Outdated

Copilot AI review requested due to automatic review settings June 19, 2026 10:29

Copilot started reviewing on behalf of efiacor June 19, 2026 10:29 View session

Copilot AI reviewed Jun 19, 2026

View reviewed changes

Comment thread .github/workflows/copilot-review.yaml

Comment thread .github/workflows/copilot-review.yaml

Comment thread .github/copilot-code-review.yml

Potential fix for pull request finding

eabb946

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>

efiacor force-pushed the copilot_review_config branch from 5cf161f to eabb946 Compare June 19, 2026 10:42

liamfallon mentioned this pull request Jun 20, 2026

Check if CoPilot has reviewed a PR so that we can turn off automatic CoPilot reviews #1061

Closed

8 tasks

liamfallon previously approved these changes Jun 22, 2026

View reviewed changes

dosubot Bot added the lgtm #ededed label Jun 22, 2026

mozesl-nokia reviewed Jun 22, 2026

View reviewed changes

Comment thread .github/copilot-code-review.yml Outdated

Comment thread .github/copilot-code-review.yml

Address review comments

ec34ebc

Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>

Copilot AI review requested due to automatic review settings June 22, 2026 14:09

efiacor dismissed liamfallon’s stale review via ec34ebc June 22, 2026 14:09

Copilot started reviewing on behalf of efiacor June 22, 2026 14:09 View session

Copilot AI reviewed Jun 22, 2026

View reviewed changes

Comment thread .github/copilot-code-review.yml

Potential fix for pull request finding

94c909f

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>

Copilot AI review requested due to automatic review settings June 22, 2026 14:38

Copilot started reviewing on behalf of efiacor June 22, 2026 14:39 View session

Copilot AI reviewed Jun 22, 2026

View reviewed changes

Comment thread .github/workflows/copilot-review.yaml Outdated

aravindtga previously approved these changes Jun 22, 2026

View reviewed changes

Potential fix for pull request finding

c27cd7b

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>

efiacor dismissed aravindtga’s stale review via c27cd7b June 22, 2026 18:04

efiacor requested review from Copilot, liamfallon and mozesl-nokia June 22, 2026 18:04

Copilot started reviewing on behalf of efiacor June 22, 2026 18:05 View session

efiacor requested a review from aravindtga June 22, 2026 18:05

Copilot AI reviewed Jun 22, 2026

View reviewed changes

CsatariGergely approved these changes Jun 23, 2026

View reviewed changes

liamfallon approved these changes Jun 23, 2026

View reviewed changes

liamfallon merged commit 6573861 into kptdev:main Jun 23, 2026
30 of 32 checks passed

efiacor deleted the copilot_review_config branch June 24, 2026 08:42

Uh oh!

Conversation

efiacor commented Jun 19, 2026

Description

Related Issue(s)

Type of Change

Checklist

Testing Instructions

Additional Notes

AI Disclosure

Uh oh!

netlify Bot commented Jun 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for kpt-porch ready!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

liamfallon left a comment

Choose a reason for hiding this comment

Uh oh!

liamfallon commented Jun 22, 2026

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

sonarqubecloud Bot commented Jun 22, 2026

Quality Gate passed

Uh oh!

CsatariGergely Jun 23, 2026

Choose a reason for hiding this comment

Uh oh!

efiacor Jun 23, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

netlify Bot commented Jun 19, 2026 •

edited

Loading