From 013d531101d172afeffc6d907ccb60f26bb4dc70 Mon Sep 17 00:00:00 2001 From: Sam Morrow Date: Wed, 15 Apr 2026 23:36:25 +0200 Subject: [PATCH] chore: remove unused newTestGitClient function Remove the now-unused newTestGitClient helper that was left behind after refactoring tests to use initGitRepo directly. This fixes the golangci-lint 'unused' error in CI. Co-authored-by: BagToad <47394200+BagToad@users.noreply.github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- pkg/cmd/skills/publish/publish_test.go | 7 ------- 1 file changed, 7 deletions(-) diff --git a/pkg/cmd/skills/publish/publish_test.go b/pkg/cmd/skills/publish/publish_test.go index 4bfd05981..6da0e01a4 100644 --- a/pkg/cmd/skills/publish/publish_test.go +++ b/pkg/cmd/skills/publish/publish_test.go @@ -20,13 +20,6 @@ import ( "github.com/stretchr/testify/require" ) -func newTestGitClient(t *testing.T, remoteURLs map[string]string) *git.Client { - t.Helper() - dir := t.TempDir() - initGitRepo(t, dir, remoteURLs) - return &git.Client{RepoDir: dir} -} - // initGitRepo initializes a git repo in the given directory and adds remotes. // Use this when the git repo must live in the same directory as the skill files. func initGitRepo(t *testing.T, dir string, remoteURLs map[string]string) {