Skip to content

Comments

docs: add inline review comments check to PR workflow skill#469

Merged
ejc3 merged 1 commit intomainfrom
pr-skill-inline-comments
Feb 23, 2026
Merged

docs: add inline review comments check to PR workflow skill#469
ejc3 merged 1 commit intomainfrom
pr-skill-inline-comments

Conversation

@ejc3
Copy link
Owner

@ejc3 ejc3 commented Feb 23, 2026

Step 1 of the PR workflow now reads both PR-level comments AND inline code review comments before any PR operation. Inline comments contain file-specific findings that are often more actionable than summary comments.

Step 1 now reads both PR-level comments AND inline code review comments
(gh api .../pulls/<pr>/comments) before any PR operation. Inline comments
contain file-specific findings that are often more actionable than summary
comments.
@ejc3 ejc3 merged commit 3e02204 into main Feb 23, 2026
@claude-claude
Copy link
Contributor

claude-claude bot commented Feb 23, 2026

🔍 Claude Review

SEVERITY: none

Findings

No issues found. This is a clean, well-scoped documentation change.

  • The PR description accurately reflects the single commit's changes
  • The gh api command correctly uses .line // .original_line to handle both added-line and deleted-line inline comments
  • The {owner}/{repo} placeholder style is consistent with the existing <pr-number> convention in the same file
  • The explanatory text about inline comments being "often MORE important" provides useful guidance

Summary

Clean docs-only PR that adds inline code review comment retrieval to the PR workflow skill's mandatory Step 1. No issues found.


Review by Claude | Run

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