Skip to content

Commit 9f62080

Browse files
committed
fix(supply): label framework install version as 'manifest default'
Copilot review: '(default: X)' could read as 'the buildpack installs this by default'. Say 'manifest default' — for java-cfenv the manifest default differs from the version actually installed (resolved per Spring Boot major).
1 parent 6d0b71d commit 9f62080

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

src/java/supply/supply.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,5 +181,7 @@ func (s *Supplier) frameworkVersionSuffix(framework frameworks.Framework) string
181181
return ""
182182
}
183183

184-
return fmt.Sprintf(" (default: %s)", dependency.Version)
184+
// "manifest default" — the default version listed in manifest.yml, not necessarily what
185+
// gets installed (e.g. java-cfenv resolves to a Spring-Boot-major-specific version).
186+
return fmt.Sprintf(" (manifest default: %s)", dependency.Version)
185187
}

0 commit comments

Comments
 (0)