Skip to content

Support default and canonical remote for PR/MR reviews - #10

Merged
simo5 merged 1 commit into
mainfrom
review_branches
Oct 23, 2026
Merged

simo5 merged 1 commit into
mainfrom
review_branches

Conversation

@simo5

@simo5 simo5 commented Oct 23, 2026

Copy link
Copy Markdown
Owner

Add support for g:vimini_default_remote and implement intelligent remote resolution for GitHub PR and GitLab MR reviews. When no remote is explicitly specified, Vimini now checks user configuration, falls back to origin if available, or discovers the canonical remote for the current branch and repository.

Assisted-by: Gemini:Gemini 3.8 Flash

@simo5
simo5 force-pushed the review_branches branch 3 times, most recently from fc94986 to aa5320f Compare October 23, 2026 20:14
Add automatic git remote resolution when reviewing pull or merge
requests without an explicit remote specified. The resolver checks
g:vimini_default_remote, prioritizes 'upstream' in fork environments,
inspects canonical branch tracking configurations, and falls back to
'origin'. Also pass '--' in git fetch commands to ensure positional
arguments are handled safely.

Assisted-by: Gemini:Gemini 3.8 Flash
Signed-off-by: Simo Sorce <simo@redhat.com>
@simo5
simo5 merged commit e88650f into main Oct 23, 2026
1 check passed
@simo5
simo5 deleted the review_branches branch October 23, 2026 20:26
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