Skip to content

Commit

Permalink
Add another guard to :Gbrowse support
Browse files Browse the repository at this point in the history
Now that I've actually had time to assess things, it's clear this
parameter is only used as a way to convey the branch name and makes zero
sense.  So, it's going to get dropped sooner or later.
  • Loading branch information
tpope committed Dec 1, 2015
1 parent bcddf09 commit 28c4895
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion autoload/rhubarb.vim
Expand Up @@ -184,7 +184,7 @@ endfunction
" Section: Fugitive :Gbrowse support

function! rhubarb#fugitive_url(opts, ...) abort
if a:0 || type(a:opts) != type({}) || !has_key(a:opts, 'repo')
if a:0 || type(a:opts) != type({}) || !has_key(a:opts, 'repo') || !has_key(a:opts, 'revision')
return ''
endif
let root = s:homepage_for_url(get(a:opts, 'remote'))
Expand Down

0 comments on commit 28c4895

Please sign in to comment.