Do not load config for the bash completion command

When a user is sourcing porter's completion command in their zshrc file
for example, if there is bad config, this causes their entire shell to
not load properly. Also the command doesn't need any configuration so
loading is slow and silly.

Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>
Co-authored-by: Yingrong Zhao <yingrong.zhao@gmail.com>

cmd/porter/completion.go

@@ -30,7 +30,8 @@ For additional details see: https://porter.sh/install#command-completion`,
 		},
 	}
 	cmd.Annotations = map[string]string{
-		"group": "meta",
+		"group":    "meta",
+		skipConfig: "",
 	}
 	return cmd
 }

cmd/porter/completion_test.go

@@ -5,6 +5,7 @@ import (
 	"os"
 	"testing"
 
+	"get.porter.sh/porter/pkg/porter"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 )
@@ -24,3 +25,10 @@ func TestCompletion(t *testing.T) {
 	// Test the output of the command contains a specific string for bash.
 	assert.Contains(t, out.String(), "bash completion for porter")
 }
+
+func TestCompletion_SkipConfig(t *testing.T) {
+	p := porter.NewTestPorter(t)
+	cmd := buildCompletionCommand(p.Porter)
+	shouldSkip := shouldSkipConfig(cmd)
+	require.True(t, shouldSkip, "expected that we skip loading configuration for the completion command")
+}