[NO-REF] Update Claude PR Review workflow to remove deprecated arguments#162
[NO-REF] Update Claude PR Review workflow to remove deprecated arguments#162
Conversation
Code Review SummaryThis is a simple and clean maintenance PR that removes two deprecated arguments ( Detailed Feedback
ConclusionThis is a straightforward housekeeping change with no logic impact on the Android client itself. The motivation is well-documented in the PR description and the referenced upstream PR. Approved — good to merge. |
There was a problem hiding this comment.
Pull request overview
Updates the repository’s GitHub Actions configuration for the “Claude PR Review” reusable workflow by removing deprecated inputs so the workflow no longer fails and continues producing inline review comments via the new default behavior.
Changes:
- Removed the deprecated
hide_previous_commentsinput from the Claude PR review workflow invocation.
💡 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.
PR Description
This PR adds the following changes:
hide_previous_commentsinline_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: