Rename PR diff actions to review #82

Closed
opened 2026-04-07 17:04:11 +00:00 by barrettruth · 1 comment
barrettruth commented 2026-04-07 17:04:11 +00:00

Summary

The PR picker currently exposes review as a diff action. That undersells what the feature is trying to be and keeps the plugin's terminology inconsistent.

Goals

  • rename the PR review action from diff to review
  • align docs, picker labels, and help text around review terminology
  • make PR review feel like a first-class workflow instead of a thin diff helper

Scope

  • picker action labels and descriptions
  • README and vimdoc wording
  • any user-facing references to PR diff that are really review semantics

Notes

This should be terminology and UX cleanup, not a redesign of the underlying review engine.

Part of #63.

## Summary The PR picker currently exposes review as a `diff` action. That undersells what the feature is trying to be and keeps the plugin's terminology inconsistent. ## Goals - rename the PR review action from `diff` to `review` - align docs, picker labels, and help text around review terminology - make PR review feel like a first-class workflow instead of a thin diff helper ## Scope - picker action labels and descriptions - README and vimdoc wording - any user-facing references to PR `diff` that are really review semantics ## Notes This should be terminology and UX cleanup, not a redesign of the underlying review engine. Part of #63.
barrettruth commented 2026-04-08 18:22:28 +00:00

Closing as completed. PR review is now the first-class term in the picker and command surface, and :Forge pr diff is kept only as a legacy alias for :Forge pr review. That matches the scope described here.

Closing as completed. PR review is now the first-class term in the picker and command surface, and :Forge pr diff is kept only as a legacy alias for :Forge pr review. That matches the scope described here.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
barrettruth/forge.nvim#82
No description provided.