release() with detached Git head doesn't throw error

[krlmlr]

@stewid: Can this be done better? I have found is_commit(), but not is_branch().

[stewid]

@krlmlr Looks fine 👍 . I have added the is_branch method to git2r ropensci/git2r@a5e65c3

I suggest that you add n = 1 in c1 <- git2r::commits(r, n = 1)[[1]]
https://github.com/krlmlr/devtools/blob/feature/git-detached/R/git.R#L32 to avoid listing all commits

[krlmlr]

Thanks. The code is now using branch_target(), looks like this is the most accurate solution (equivalent to commits(r, n = 1, time = FALSE, topological = FALSE)).

[hadley]

Thanks! Do you have any thoughts on #1128?