Skip to content

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

Closed
liamfallon wants to merge 9 commits into
kptdev:mainfrom
Nordix:pr-review-checks
Closed

Check if CoPilot has reviewed a PR so that we can turn off automatic CoPilot reviews#1061
liamfallon wants to merge 9 commits into
kptdev:mainfrom
Nordix:pr-review-checks

Conversation

@liamfallon

Copy link
Copy Markdown
Contributor

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


Description

  • What changed: Add this github action to test if a CoPolit review has been performed and if all comments are resolved
  • Why it’s needed: CoPilot automatic checking is crazy
  • How it works: Github action checks if the PR has been reviewd by Copilot and if all comments are resolved.

Type of Change

  • Bug fix
  • New feature
  • Enhancement
  • Refactor
  • Documentation
  • Tests
  • Other: CI

AI Disclosure

  • I have used AI in the creation of this PR.

Github action created by Amazon Q

Signed-off-by: liamfallon <liam.fallon@est.tech>
@liamfallon liamfallon requested review from a team and Copilot June 18, 2026 15:48
@netlify

netlify Bot commented Jun 18, 2026

Copy link
Copy Markdown

Deploy Preview for kpt-porch ready!

Name Link
🔨 Latest commit fb26b32
🔍 Latest deploy log https://app.netlify.com/projects/kpt-porch/deploys/6a341b25002e110008a32de0
😎 Deploy Preview https://deploy-preview-1061--kpt-porch.netlify.app
📱 Preview on mobile
Toggle QR Code...

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.

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

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

Adds a GitHub Actions workflow intended to gate PR progress on (1) a GitHub Copilot review having occurred and (2) all review threads being resolved, to reduce/disable noisy automatic Copilot reviews.

Changes:

  • Introduces .github/workflows/pr-review-checks.yaml to run on PR and review/comment events.
  • Uses actions/github-script to detect a Copilot-authored review and to check reviewThreads.isResolved via GraphQL.

Comment thread .github/workflows/pr-review-checks.yaml Outdated
Comment thread .github/workflows/pr-review-checks.yaml
Comment thread .github/workflows/pr-review-checks.yaml Outdated
Comment thread .github/workflows/pr-review-checks.yaml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Liam Fallon <35595825+liamfallon@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 18, 2026 15:54

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 7 comments.

Comment thread .github/workflows/pr-review-checks.yaml
Comment thread .github/workflows/pr-review-checks.yaml
Comment thread .github/workflows/pr-review-checks.yaml
Comment thread .github/workflows/pr-review-checks.yaml Outdated
Comment thread .github/workflows/pr-review-checks.yaml Outdated
Comment thread .github/workflows/pr-review-checks.yaml Outdated
Comment thread .github/workflows/pr-review-checks.yaml Outdated
Signed-off-by: liamfallon <liam.fallon@est.tech>
Signed-off-by: liamfallon <liam.fallon@est.tech>
Copilot AI review requested due to automatic review settings June 18, 2026 16:04

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 6 comments.

Comment thread .github/workflows/pr-review-checks.yaml Outdated
Comment thread .github/workflows/pr-review-checks.yaml
Comment thread .github/workflows/pr-review-checks.yaml Outdated
Comment thread .github/workflows/pr-review-checks.yaml Outdated
Comment thread .github/workflows/pr-review-checks.yaml
Comment thread .github/workflows/pr-review-checks.yaml
Signed-off-by: liamfallon <liam.fallon@est.tech>
Signed-off-by: liamfallon <liam.fallon@est.tech>
Copilot AI review requested due to automatic review settings June 18, 2026 16:10

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

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

Comment thread .github/workflows/pr-review-checks.yaml Outdated
Comment thread .github/workflows/pr-review-checks.yaml
Signed-off-by: liamfallon <liam.fallon@est.tech>
Signed-off-by: liamfallon <liam.fallon@est.tech>
Copilot AI review requested due to automatic review settings June 18, 2026 16:16

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

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

Comment thread .github/workflows/pr-review-checks.yaml
Comment thread .github/workflows/pr-review-checks.yaml
Comment thread .github/workflows/list-pr-reviews.yaml Outdated
Comment thread .github/workflows/list-pr-reviews.yaml Outdated
Signed-off-by: liamfallon <liam.fallon@est.tech>
@sonarqubecloud

Copy link
Copy Markdown

@liamfallon

Copy link
Copy Markdown
Contributor Author

Superseded by #1064

@liamfallon liamfallon closed this Jun 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants