Skip to content

Add rereview-pr skill for follow-up PR reviews - #494

Merged
LarsLaskowski merged 1 commit into
mainfrom
claude/adopt-rereview-pr-skill-85bpbl
Jul 26, 2026
Merged

Add rereview-pr skill for follow-up PR reviews#494
LarsLaskowski merged 1 commit into
mainfrom
claude/adopt-rereview-pr-skill-85bpbl

Conversation

@LarsLaskowski

@LarsLaskowski LarsLaskowski commented Jul 26, 2026

Copy link
Copy Markdown
Owner

Summary

Adds a new rereview-pr skill that enables focused re-review of pull requests after review feedback has been addressed. The skill checks whether previous findings were resolved, evaluates only the changes made since the last review, and verifies CI/SonarQube status—without modifying any code.

Updated references in CLAUDE.md and .github/copilot-instructions.md to include the new skill in the list of available skills.

Linked issues

Review notes

The skill is read-only and follows the same non-invasive pattern as review-pr. It narrows scope to:

  • Resolution status of previous findings
  • Code changes since the last review point
  • PR description accuracy
  • CI and SonarQube Cloud status only

The skill explicitly forbids code modifications, commits, or pushes, and does not post to GitHub unless explicitly requested by the user.

Adopts the rereview-pr skill from DockerUpdateGuard and adapts it to this
repository: the F1-Telemetry repository name, the F1Server.Tests project, the
review criteria used by the existing review-pr skill, and the CI check names
of this repository (Build, Test & Analyze plus the SonarQube Cloud quality
gate). The skill list in CLAUDE.md and copilot-instructions.md is updated
accordingly.
@sonarqubecloud

Copy link
Copy Markdown

@LarsLaskowski
LarsLaskowski merged commit 76864c3 into main Jul 26, 2026
6 checks passed
@LarsLaskowski
LarsLaskowski deleted the claude/adopt-rereview-pr-skill-85bpbl branch July 26, 2026 07:28
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.

2 participants