Apply suggestions from code review

Co-authored-by: Owen Mansel-Chan <62447351+owen-mc@users.noreply.github.com>
This commit is contained in:
Michael B. Gale
2026-02-03 10:29:15 +00:00
parent d079671ec8
commit d5c4a19efa
2 changed files with 5 additions and 4 deletions

View File

@@ -570,10 +570,11 @@ func EmitExtractionFailedForProjects(path []string) {
}
func EmitPrivateRegistryUsed(writer DiagnosticsWriter, configs []string) {
lines := []string{}
n := len(configs)
lines := make([]string, n)
for i := range configs {
lines = append(lines, fmt.Sprintf("* %s", configs[i]))
lines[i] = fmt.Sprintf("* %s", configs[i])
}
emitDiagnosticTo(
@@ -582,7 +583,7 @@ func EmitPrivateRegistryUsed(writer DiagnosticsWriter, configs []string) {
"Go extraction used private package registries",
fmt.Sprintf(
"Go was extracted using the following private package registr%s:\n\n%s\n",
plural(len(lines), "y", "ies"),
plural(n, "y", "ies"),
strings.Join(lines, "\n")),
severityNote,
fullVisibility,

View File

@@ -112,7 +112,7 @@ func getEnvVars() []string {
if err != nil {
slog.Error("Unable to parse proxy configurations", slog.String("error", err.Error()))
} else {
activeConfigs := []RegistryConfig{}
activeConfigs := make([]RegistryConfig, 0, len(val))
// We only care about private registry configurations that are relevant to Go and
// filter others out at this point.