-
Notifications
You must be signed in to change notification settings - Fork 26
Add PR Copilot Review workflow and normalize review result script #26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
jacwu
commented
Sep 26, 2025
- Create a new GitHub Actions workflow for PR Copilot Review.
- Implement steps to compute diffs, prepare fallback reviews, and run the Copilot CLI for code reviews.
- Update the normalization script to read from the new raw review output file instead of the previous codex review file.
- Create a new GitHub Actions workflow for PR Copilot Review. - Implement steps to compute diffs, prepare fallback reviews, and run the Copilot CLI for code reviews. - Update the normalization script to read from the new raw review output file instead of the previous codex review file.
Reviewed by GitHub Copilot CLI |
Reviewed by Codex |
There was a problem hiding this 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 adds a new GitHub Actions workflow for Copilot-based PR reviews and standardizes output handling between the existing Codex and new Copilot workflows.
- Add new GitHub Actions workflow for PR Copilot Review that parallels the existing Codex review functionality
- Normalize file naming conventions by using
raw_review.mdas the standard intermediate output file across both workflows - Update the normalization script to read from the standardized filename
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.github/workflows/pr-copilot-review.yml |
New workflow implementing Copilot CLI-based code reviews with similar structure to existing Codex workflow |
.github/workflows/pr-codex-review.yml |
Updated to use standardized raw_review.md filename and improved environment variable handling |
scripts/normalize_review_result.py |
Updated to read from raw_review.md instead of codex_review.md for consistency |
| env: | ||
| GITHUB_TOKEN: ${{ secrets.COPILOT_CLI_PAT }} |
Copilot
AI
Oct 9, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent indentation - the environment variable should be indented with 2 spaces to align with the step structure, not 4 spaces.
|
|
||
| - name: Run Copilot code review | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.COPILOT_CLI_PAT }} |
Copilot
AI
Oct 9, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent indentation - the environment variable should be indented with 2 spaces to align with the step structure, not 4 spaces.
| GITHUB_TOKEN: ${{ secrets.COPILOT_CLI_PAT }} | |
| GITHUB_TOKEN: ${{ secrets.COPILOT_CLI_PAT }} |