Add i18n for Command Log

[jesseduffield]

We need to internationalise the strings we pass to the LogCommand function. Due to an oversight, these are all hard-coded at the moment:

Here's an example of i18n'ing one of the instances:

diff --git a/pkg/commands/git_commands/rebase.go b/pkg/commands/git_commands/rebase.go
index 83c3fbe09..3ad7b7b32 100644
--- a/pkg/commands/git_commands/rebase.go
+++ b/pkg/commands/git_commands/rebase.go
@@ -149,7 +149,13 @@ func (self *RebaseCommands) InteractiveRebase(commits []*models.Commit, index in
 }
 
 func (self *RebaseCommands) EditRebase(branchRef string) error {
-	self.os.LogCommand(fmt.Sprintf("Beginning interactive rebase at '%s'", branchRef), false)
+	msg := utils.ResolvePlaceholderString(
+		self.Tr.Actions.BeginningInteractiveRebaseAtRef,
+		map[string]string{
+			"ref": branchRef,
+		},
+	)
+	self.os.LogCommand(msg, false)
 	return self.PrepareInteractiveRebaseCommand(PrepareInteractiveRebaseCommandOpts{
 		baseShaOrRoot: branchRef,
 		instruction:   daemon.NewInsertBreakInstruction(),
diff --git a/pkg/i18n/english.go b/pkg/i18n/english.go
index ecfa6a2ae..e693f7715 100644
--- a/pkg/i18n/english.go
+++ b/pkg/i18n/english.go
@@ -710,6 +710,7 @@ type Actions struct {
 	BisectMark                        string
 	RemoveWorktree                    string
 	AddWorktree                       string
+	BeginningInteractiveRebaseAtRef   string
 }
 
 const englishIntroPopupMessage = `
@@ -1424,6 +1425,7 @@ func EnglishTranslationSet() TranslationSet {
 			BisectMark:                        "Bisect mark",
 			RemoveWorktree:                    "Remove worktree",
 			AddWorktree:                       "Add worktree",
+			BeginningInteractiveRebaseAtRef:   "Beginning interactive rebase at '{{.ref}}'",
 		},
 		Bisect: Bisect{
 			Mark:                        "Mark current commit (%s) as %s",

[KarlHeitmann]

I want to do this one. Any thoughts on how to test it? Or it is not necessary: just move the code from hard coded to i18n file

[jesseduffield]

Sounds good @KarlHeitmann ! No need to worry about tests, I consider this copy stuff low-stakes

[topher200]

I think this issue can be closed -- it has been resolved by #2852