Skip to content

Commit

Permalink
support detached heads when showing the selected branch
Browse files Browse the repository at this point in the history
  • Loading branch information
jesseduffield committed Nov 21, 2019
1 parent 6b7aaec commit 55ff0c0
Show file tree
Hide file tree
Showing 5 changed files with 39 additions and 7 deletions.
15 changes: 13 additions & 2 deletions pkg/commands/git.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import (
"os"
"os/exec"
"path/filepath"
"regexp"
"strings"
"time"

Expand All @@ -21,6 +22,13 @@ import (
gogit "gopkg.in/src-d/go-git.v4"
)

// this takes something like:
// * (HEAD detached at 264fc6f5)
// remotes
// and returns '264fc6f5' as the second match

const CurrentBranchNameRegex = `(?m)^\*.*?([^ ]*?)\)?$`

func verifyInGitRepo(runCmd func(string) error) error {
return runCmd("git status")
}
Expand Down Expand Up @@ -325,8 +333,11 @@ func (c *GitCommand) NewBranch(name string) error {
// CurrentBranchName is a function.
func (c *GitCommand) CurrentBranchName() (string, error) {
branchName, err := c.OSCommand.RunCommandWithOutput("git symbolic-ref --short HEAD")
if err != nil {
branchName, err = c.OSCommand.RunCommandWithOutput("git rev-parse --short HEAD")
if err != nil || branchName == "HEAD\n" {
output, err := c.OSCommand.RunCommandWithOutput("git branch --contains")
re := regexp.MustCompile(CurrentBranchNameRegex)
match := re.FindStringSubmatch(output)
branchName = match[1]
if err != nil {
return "", err
}
Expand Down
7 changes: 4 additions & 3 deletions pkg/commands/loading_remotes.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,14 +22,15 @@ func (c *GitCommand) GetRemotes() ([]*Remote, error) {
// first step is to get our remotes from go-git
remotes := make([]*Remote, len(goGitRemotes))
for i, goGitRemote := range goGitRemotes {
name := goGitRemote.Config().Name
remoteName := goGitRemote.Config().Name

re := regexp.MustCompile(fmt.Sprintf("%s\\/(.*)", name))
re := regexp.MustCompile(fmt.Sprintf("%s\\/(.*)", remoteName))
matches := re.FindAllStringSubmatch(remoteBranchesStr, -1)
branches := make([]*RemoteBranch, len(matches))
for j, match := range matches {
branches[j] = &RemoteBranch{
Name: match[1],
Name: match[1],
RemoteName: remoteName,
}
}

Expand Down
5 changes: 3 additions & 2 deletions pkg/commands/remote_branch.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,9 @@ import (

// Remote Branch : A git remote branch
type RemoteBranch struct {
Name string
Selected bool
Name string
Selected bool
RemoteName string
}

// GetDisplayStrings returns the display string of branch
Expand Down
8 changes: 8 additions & 0 deletions pkg/gui/keybindings.go
Original file line number Diff line number Diff line change
Expand Up @@ -1057,6 +1057,14 @@ func (gui *Gui) GetInitialKeybindings() []*Binding {
Handler: gui.handleRemoveRemote,
Description: gui.Tr.SLocalize("removeRemote"),
},
{
ViewName: "branches",
Contexts: []string{"remote-branches"},
Key: gocui.KeySpace,
Modifier: gocui.ModNone,
Handler: gui.handleCheckoutRemoteBranch,
Description: gui.Tr.SLocalize("checkout"),
},
{
ViewName: "commits",
Key: gocui.MouseLeft,
Expand Down
11 changes: 11 additions & 0 deletions pkg/gui/remote_branches_panel.go
Original file line number Diff line number Diff line change
Expand Up @@ -81,3 +81,14 @@ func (gui *Gui) renderRemoteBranchesWithSelection() error {

return nil
}

func (gui *Gui) handleCheckoutRemoteBranch(g *gocui.Gui, v *gocui.View) error {
remoteBranch := gui.getSelectedRemoteBranch()
if remoteBranch == nil {
return nil
}
if err := gui.handleCheckoutBranch(remoteBranch.RemoteName + "/" + remoteBranch.Name); err != nil {
return err
}
return gui.switchBranchesPanelContext("local-branches")
}

0 comments on commit 55ff0c0

Please sign in to comment.