unescape another string

when it contains percentages.

This is a really strange one. It's a linting warning in my editor
and it doesn't stop me from compiling, but it breaks `go test`.

A basic file to reproduce what I'm talking about:

package main

import "fmt"

func main() {
	notSprintf("test %s") // compiler complains here thinking %s needs a corresponding argument
}

func notSprintf(formatStr string, formatArgs ...interface{}) string {
	if formatArgs != nil {
		return formatStr
	}
	return fmt.Sprintf(formatStr, formatArgs...)
}

pkg/commands/loading_remotes.go

@@ -9,7 +9,8 @@ import (
 
 func (c *GitCommand) GetRemotes() ([]*Remote, error) {
 	// get remote branches
-	remoteBranchesStr, err := c.OSCommand.RunCommandWithOutput("git for-each-ref --format='%%(refname:strip=2)' refs/remotes")
+	unescaped := "git for-each-ref --format='%(refname:strip=2)' refs/remotes"
+	remoteBranchesStr, err := c.OSCommand.RunCommandWithOutput(unescaped)
 	if err != nil {
 		return nil, err
 	}

pkg/commands/os.go

@@ -59,8 +59,11 @@ func (c *OSCommand) SetCommand(cmd func(string, ...string) *exec.Cmd) {
 }
 
 // RunCommandWithOutput wrapper around commands returning their output and error
-// NOTE: because this takes a format string followed by format args, you'll need
-// to escape any percentage signs via '%%'.
+// NOTE: If you don't pass any formatArgs we'll just use the command directly,
+// however there's a bizarre compiler error/warning when you pass in a formatString
+// with a percent sign because it thinks it's supposed to be a formatString when
+// in that case it's not. To get around that error you'll need to define the string
+// in a variable and pass the variable into RunCommandWithOutput.
 func (c *OSCommand) RunCommandWithOutput(formatString string, formatArgs ...interface{}) (string, error) {
 	command := formatString
 	if formatArgs != nil {

pkg/gui/keybindings.go

@@ -424,7 +424,7 @@ func (gui *Gui) GetInitialKeybindings() []*Binding {
 			Key:         'P',
 			Modifier:    gocui.ModNone,
 			Handler:     gui.handlePushTag,
-			Description: gui.Tr.SLocalize("pushTags"),
+			Description: gui.Tr.SLocalize("pushTag"),
 		},
 		{
 			ViewName:    "branches",

pkg/i18n/english.go

@@ -904,8 +904,8 @@ func addEnglish(i18nObject *i18n.Bundle) error {
 			ID:    "PushTagTitle",
 			Other: "remote to push tag '{{.tagName}}' to:",
 		}, &i18n.Message{
-			ID:    "pushTags",
-			Other: "push tags",
+			ID:    "pushTag",
+			Other: "push tag",
 		}, &i18n.Message{
 			ID:    "createTag",
 			Other: "create tag",