Skip to content

Improve PR review instructions - #334

Merged
eshulman2 merged 1 commit into
mainfrom
docs/improve-pr-review-instructions
Sep 1, 2026
Merged

Improve PR review instructions#334
eshulman2 merged 1 commit into
mainfrom
docs/improve-pr-review-instructions

Conversation

@eshulman2

@eshulman2 eshulman2 commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • improve the informational comment Forge posts directly on newly created GitHub pull requests
  • explain how to submit review feedback through a Request changes review
  • clarify that Forge automatically addresses submitted review feedback
  • clarify that regular PR comments remain available for conversation and Forge commands

Testing

  • uv run pytest tests/unit/workflow/nodes/test_pr_creation_informational_comment.py (3 passed)
  • uv run ruff check src/forge/workflow/nodes/pr_creation.py tests/unit/workflow/nodes/test_pr_creation_informational_comment.py

@eshulman2
eshulman2 force-pushed the docs/improve-pr-review-instructions branch from 2147969 to 4868c31 Compare September 1, 2026 12:16
@eshulman2

Copy link
Copy Markdown
Collaborator Author

🤖 Reviewing a Forge pull request

To send code feedback to Forge, open Files changed, select Review changes, and submit a Request changes review. Forge will address the review feedback automatically and push updates to this pull request.

Regular comments are ignored by the review workflow. Use them for conversation or the Forge commands below:

  • /forge rebase - Merge the base branch (e.g. main) into this PR branch, with conflicts resolved by AI.
  • /forge skip-gate <name> - Skip a named CI check (substring match) for this PR. This setting persists across subsequent pushes.
  • /forge unskip-gate <name> - Remove a previously set CI check skip.

@eshulman2
eshulman2 merged commit bdbce62 into main Sep 1, 2026
8 checks passed
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