add a comment explaining why we build an sshUsers map

znull · znull · commit 4a6887f99e0b · 2021-12-16T17:00:38.000-07:00
diff --git a/pkg/cmd/codespace/ssh.go b/pkg/cmd/codespace/ssh.go
@@ -194,6 +194,10 @@ func (a *App) printOpenSSHConfig(ctx context.Context, opts configOptions) error
 		return err
 	}
 
+	// store a mapping of repository -> remote ssh username. This is
+	// necessary because the username can vary between codespaces, but
+	// since fetching it is slow, we store it here so we at least only do
+	// it once per repository.
 	sshUsers := map[string]string{}
 
 	for _, cs := range codespaces {

Original file line number	Diff line number	Diff line change
`@@ -194,6 +194,10 @@ func (a *App) printOpenSSHConfig(ctx context.Context, opts configOptions) error`
`194`	`194`	`return err`
`195`	`195`	`}`
`196`	`196`
	`197`	`+ // store a mapping of repository -> remote ssh username. This is`
	`198`	`+ // necessary because the username can vary between codespaces, but`
	`199`	`+ // since fetching it is slow, we store it here so we at least only do`
	`200`	`+ // it once per repository.`
`197`	`201`	`sshUsers := map[string]string{}`
`198`	`202`
`199`	`203`	`for _, cs := range codespaces {`