git: Drop unused SafeCmdWithoutRepo() function

The `git.SafeCmdWithoutRepo()` function is not used anymore, so let's
drop it.

2 files changed, 0 insertions, 25 deletions

diff --git a/internal/git/safecmd.go b/internal/git/safecmd.go
index 85adbc405..fa5c8dd88 100644
--- a/internal/git/safecmd.go
+++ b/internal/git/safecmd.go
@@ -436,27 +436,6 @@ func SafeStdinCmd(ctx context.Context, repo repository.GitRepo, globals []Global
 	return NewCommandFactory().unsafeStdinCmd(ctx, cc.env, repo, args...)
 }
 
-// SafeCmdWithoutRepo works like Command but without a git repository. It
-// validates the arguments in the command before executing.
-func SafeCmdWithoutRepo(ctx context.Context, globals []GlobalOption, sc Cmd, opts ...CmdOpt) (*command.Command, error) {
-	cc := &cmdCfg{}
-
-	if err := handleOpts(ctx, sc, cc, opts); err != nil {
-		return nil, err
-	}
-
-	args, err := combineArgs(globals, sc, cc)
-	if err != nil {
-		return nil, err
-	}
-
-	return NewCommandFactory().unsafeBareCmd(ctx, cmdStream{
-		In:  cc.stdin,
-		Out: cc.stdout,
-		Err: cc.stderr,
-	}, cc.env, args...)
-}
-
 func combineArgs(globals []GlobalOption, sc Cmd, cc *cmdCfg) (_ []string, err error) {
 	var args []string
 
diff --git a/internal/git/safecmd_test.go b/internal/git/safecmd_test.go
index 7981b4f56..f629ca728 100644
--- a/internal/git/safecmd_test.go
+++ b/internal/git/safecmd_test.go
@@ -349,10 +349,6 @@ func TestSafeCmdValid(t *testing.T) {
 			// ignore first indeterministic arg (executable path)
 			require.Equal(t, tt.expectArgs, cmd.Args()[1:])
 
-			cmd, err = SafeCmdWithoutRepo(ctx, tt.globals, tt.subCmd, opts...)
-			require.NoError(t, err)
-			require.Equal(t, tt.expectArgs, cmd.Args()[1:])
-
 			cmd, err = SafeBareCmdInDir(ctx, testRepoPath, tt.globals, tt.subCmd, opts...)
 			require.NoError(t, err)
 			require.Equal(t, tt.expectArgs, cmd.Args()[1:])