Skip to content

docs(pr-review): clarify README migration + custom skill guidance#121

Open
srxly888-creator wants to merge 1 commit intoOpenHands:mainfrom
srxly888-creator:fix/pr-review-readme-clarity
Open

docs(pr-review): clarify README migration + custom skill guidance#121
srxly888-creator wants to merge 1 commit intoOpenHands:mainfrom
srxly888-creator:fix/pr-review-readme-clarity

Conversation

@srxly888-creator
Copy link

@srxly888-creator srxly888-creator commented Mar 27, 2026

Summary

This PR applies a small, targeted README consistency fix for the PR review plugin.

Changes

  • File: plugins/pr-review/README.md
  • Clarify custom skill example to use a unique skill name (custom-codereview-guide) instead of overriding the default public code-review skill
  • Keep /codereview trigger guidance while explicitly framing this as a supplement pattern
  • Keep migration section intact, but improve consistency around extensions-era usage
  • Expand Security notes to clarify when pull_request_target is appropriate and when lower-trust/comment-only setups should prefer pull_request

Scope

  • Minimal, documentation-only change
  • No workflow logic changes in this PR

Context

@srxly888-creator
Copy link
Author

srxly888-creator commented Mar 27, 2026

Hi maintainers — I opened two small, scoped PRs that address the same PR-review migration/clarity issue from the integrator side:

These are intentionally small and non-invasive, but together they make the migration path much clearer for downstream adopters. Happy to adjust wording or split anything further if that helps review.

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.

1 participant