Skip to content

Conversation

@RayStick
Copy link
Member

@RayStick RayStick commented Sep 8, 2025

Intended behaviour:

  • The first person to add themselves as a reviewer on a PR is tagged as the main reviewer, and gets the main_rev_checklist.md in a comment.
  • Any additional reviewers who add themselves get the rev_checklist.md in a comment.

@github-actions github-actions bot added the Documentation This issue or PR is about the documentation label Sep 8, 2025
@RayStick RayStick requested a review from Copilot September 8, 2025 10:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements an automated reviewer checklist system that assigns different checklists based on reviewer order. The first person to add themselves as a reviewer gets the main reviewer checklist, while subsequent reviewers receive a supporting reviewer checklist.

  • Changes workflow trigger from PR assignment to review requests
  • Implements logic to differentiate between main and supporting reviewers based on count
  • Standardizes placeholder replacement across both checklist templates

Reviewed Changes

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

File Description
README.md Adds documentation for the new main reviewer checklist feature
.github/workflows/rev_checklist.md Creates supporting reviewer checklist template
.github/workflows/main_rev_checklist.md Standardizes placeholder name from __MAINREV__ to __REV__
.github/workflows/main-reviewer.yml Refactors workflow to handle both main and supporting reviewer checklists

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@RayStick RayStick merged commit 22b6e51 into master Sep 8, 2025
1 check passed
@RayStick RayStick deleted the demo-PR branch September 8, 2025 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation This issue or PR is about the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants