Skip to content

cmd: let current-open-PR mutators omit {num}#456

Merged
barrettruth merged 1 commit intomainfrom
feature/implicit-open-pr-mutators
Apr 27, 2026
Merged

cmd: let current-open-PR mutators omit {num}#456
barrettruth merged 1 commit intomainfrom
feature/implicit-open-pr-mutators

Conversation

@barrettruth
Copy link
Copy Markdown
Owner

Problem

:Forge pr close, approve, merge, draft, and ready required an explicit PR number even when the current branch already mapped to exactly one open PR. That made those mutators inconsistent with the existing implicit current-PR flows in :Forge pr, :Forge review, and :Forge pr ci.

Closes #452.

Solution

Allow those mutators to omit {num} and route the omitted-subject form through the existing current-open-PR resolver, with repo= and head= available for disambiguation. Update the help text and command specs so parsing, completion, dispatch, and warning behavior all match the new command surface.

Extend approve, merge, close, draft, and ready to resolve the current open PR when {num} is omitted, while preserving explicit targeting and current repo/head disambiguation behavior.
@barrettruth barrettruth merged commit ce352ee into main Apr 27, 2026
3 checks passed
@barrettruth barrettruth deleted the feature/implicit-open-pr-mutators branch April 27, 2026 17:09
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.

cmd: let current-open-PR mutators omit {num}

1 participant