Skip to content
gh-fix-pr-feedback logo

Fix PR Review Feedback

gh-fix-pr-feedback

Address PR review feedback by making changes and replying to comments. Use when a PR has open review comments needing responses.

SKILL.md

Full skill instructions

Fix PR Review Feedback

Address PR review comments by implementing fixes and responding to each comment.

When to Use

  • PR has open review comments requiring responses
  • Ready to implement reviewer's requested changes
  • Need to notify reviewers of fixes
  • PR is blocked on feedback

Quick Reference

# 1. Get all review comments
gh api repos/​OWNER/​REPO/​pulls/​PR/​comments --jq '.[] | {id: .id, path: .path, body: .body}'

# 2. Make fixes to code
# [edit files, test, format]

# 3. Commit changes
git add . && git commit -m "fix: address PR review feedback"

# 4. Reply to EACH comment
gh api repos/​OWNER/​REPO/​pulls/​PR/​comments/​COMMENT_ID/​replies \
  --method POST -f body="✅ Fixed - [brief description]"

# 5. Push and verify
git push
gh pr checks PR

Workflow

  1. Fetch review comments: List all comments requiring response
  2. Analyze feedback: Understand all requested changes
  3. Make changes: Edit code to address each comment
  4. Run tests: Verify fixes pass locally
  5. Commit changes: Create single focused commit
  6. Reply to comments: Reply to EACH comment individually (critical!)
  7. Push and verify: Push changes and check CI status

Reply Format

Keep responses SHORT and CONCISE (1 line preferred):

  • ✅ Fixed - Updated conftest.py to use real repository root
  • ✅ Fixed - Removed duplicate test file
  • ✅ Fixed - Added error handling for edge case

Critical: Two Types of Comments

PR-level comments (general timeline):

gh pr comment <pr> --body "Response"

Review comment replies (inline code feedback):

gh api repos/​OWNER/​REPO/​pulls/​PR/​comments/​COMMENT_ID/​replies \
  --method POST -f body="✅ Fixed - description"

NEVER confuse these - use the correct API for review comments.

Error Handling

ProblemSolution
Comment ID invalidVerify ID using API
Auth failureRun gh auth status
Reply not appearingCheck API endpoint syntax
CI fails after pushReview logs and fix issues

Verification

After addressing feedback:

  • All review comments have replies
  • Changes committed and pushed
  • CI checks passing
  • No new issues introduced

References

  • See CLAUDE.md for complete PR workflow
  • See /​agents/​guides/​github-review-comments.md for detailed guide

More skills from majiayu000

xiaohongshu logo
majiayu000/claude-arsenal

xiaohongshu

xiaohongshu

286 148
View
agent-task-conductor logo
majiayu000/claude-skill-registry

agent-task-conductor

Conduct multi-agent task orchestration and workflow coordination.

663 1
View
conductor-setup logo
majiayu000/claude-skill-registry

conductor-setup

Initialize project with Conductor artifacts (product definition,

663 1
View
animation-designer logo
majiayu000/claude-skill-registry

animation-designer

Expert in web animations, transitions, and motion design using Framer Motion and CSS

663 1
View
diagramming logo
majiayu000/claude-skill-registry

diagramming

Creates Mermaid and ASCII diagrams for flowcharts, architecture, ERDs, state machines, mindmaps, and more. Use when user mentions diagram, flowchart, mermaid, ASCII diagram, text diagram, terminal diagram, visualize, C4, mindmap, architecture diagram, sequence diagram, ERD, or needs visual docume...

663 1
View
h3-pg logo
majiayu000/claude-skill-registry-data

h3-pg

PostgreSQL bindings for H3 hexagonal grid system. Use when working with H3 cells in Postgres, including spatial indexing, geometry/geography integration, and raster analysis.

23 1
View
conductor-development logo
majiayu000/claude-skill-registry

conductor-development

Context-Driven Development skill for projects using Conductor. Use this skill when you detect a `conductor/` directory in the project, when working on tasks defined in a `plan.md` file, or when the user asks about tracks, specs, or plans. Automatically applies TDD workflow, tracks task completion...

663 1
View
conductor-status logo
majiayu000/claude-skill-registry

conductor-status

Display project status, active tracks, and next actions

663 1
View
dockerization logo
majiayu000/claude-skill-registry

dockerization

Official Stakpak application containerization standard operating procedure, a step-by-step guidline to properly dockerize applications. This is a rule book curated by the Stakpak Team.

663 1
View

Popular AI tools

Kaiber logo
Video

Kaiber

Generate, edit, and beat-sync AI video with leading models in one workspace.

Paid
View
Vimcal logo
Productivity

Vimcal

The world's fastest calendar for remote work

Free
View

Transform Your Design with AI Designer by ImgCreator.ai

Freemium
View
Akool AI logo
Content & writing

Akool AI

Revolutionizing Video Production with AI-Powered Creativity

Paid
View

Extend an image past the frame and let AI fill the new aspect ratio.

Freemium
View
StarByFace logo
Security

StarByFace

Discover your celebrity doppelgänger with StarByFace!

Free
View
C

ChainClarity explains 700+ crypto whitepapers in plain English, with layered summaries, comparisons, research tools, alerts, and a $4.99 Pro plan.

Freemium
View
Opus Clip logo
Coding & apps

Opus Clip

Opus.ai: Revolutionize Your Web Experience

Free
View