Skip to content

Address review feedback on Claude documentation and command files#741

Closed
Copilot wants to merge 1 commit intoinitial-claude-commandsfrom
copilot/sub-pr-740
Closed

Address review feedback on Claude documentation and command files#741
Copilot wants to merge 1 commit intoinitial-claude-commandsfrom
copilot/sub-pr-740

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 2, 2026

Addresses 14 review comments from PR #740, primarily fixing typos in CLAUDE.md and adding clarifications to command documentation files.

Changes Made

CLAUDE.md typo corrections:

  • Line 24: 'serivces' → 'services'
  • Line 62: Removed trailing space
  • Line 64: 'specfications' → 'specifications'
  • Line 75: 'lessons.m`d' → 'lessons.md'

Command documentation clarifications:

  • update-pull-request.md L20: Clarified commit_sha comes from head.sha field
  • update-pull-request.md L24-26: Added distinction between check-runs and workflow runs
  • update-pull-request.md L77-79: Added guidance on extracting thread IDs from pull_request_review_id
  • update-notes.md L18: Removed "alphabetical or" to specify logical grouping only
  • create-or-update-issue.md L22: Added specific GraphQL query for project node ID retrieval

Not changed:

  • CLAUDE.md L93 maintains explicit numbering (1, 2, 3, 4, 5) for readability, consistent with project documentation style

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@github-project-automation github-project-automation Bot moved this to To Do in Overview Feb 2, 2026
Copilot AI changed the title [WIP] Fix comments from review on previous PR Address review feedback on Claude documentation and command files Feb 2, 2026
Copilot AI requested a review from forstmeier February 2, 2026 02:53
@forstmeier
Copy link
Copy Markdown
Collaborator

@copilot why was this opened?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants