Skip to content

[NO-REF] Update Claude PR Review workflow to remove deprecated arguments#246

Merged
esezen merged 1 commit intomasterfrom
update-claude-pr-review-workflow
Mar 19, 2026
Merged

[NO-REF] Update Claude PR Review workflow to remove deprecated arguments#246
esezen merged 1 commit intomasterfrom
update-claude-pr-review-workflow

Conversation

@pedro-lb
Copy link
Contributor

PR Description

This PR adds the following changes:

  • Removes deprecated arguments from the Claude PR review workflow
    • hide_previous_comments
    • inline_comments

💡 note: We recently switched to a new version of the "Claude PR Review" workflow that does inline comments by default, in order to improve the code review process. Please refer to this PR for more context: https://github.com/Constructor-io/shared-claude-code-resources/pull/39

🕹 Demo

Once this is merged you should see that the "Claude PR Review" workflow no longer failing, and giving you inline reviews.

You'll also see a short summary in the code review body, with all discussions extracted as inline comments in the code.

Example:

image

@pedro-lb pedro-lb requested review from esezen and jjl014 as code owners March 19, 2026 14:57
Copilot AI review requested due to automatic review settings March 19, 2026 14:57
@esezen esezen merged commit 2fce87f into master Mar 19, 2026
6 of 7 checks passed
@esezen esezen deleted the update-claude-pr-review-workflow branch March 19, 2026 14:58
Copy link

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

Updates the GitHub Actions workflow that runs the reusable “Claude PR Review” workflow by removing now-deprecated input arguments, aligning this repo with the newer upstream workflow behavior (inline comments by default).

Changes:

  • Removed the with: block from the Claude PR review job.
  • Dropped the deprecated hide_previous_comments input (and no remaining deprecated inputs found in this repo’s workflows).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@constructor-claude-bedrock
Copy link

Code Review Results

✅ Strengths

Clean, minimal change that correctly removes deprecated workflow arguments to restore the Claude PR Review functionality.

🚨 Critical Issues

None.

⚠️ Important Issues

None.

💡 Suggestions

None.

Overall Assessment: ✅ Pass

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.

3 participants