Merge pull request github#15408 from smowton/smowton/admin/log-setup-go-message

smowton · web-flow · commit 43453fea5203 · 2024-01-23T11:32:38.000Z
Log advice when a newer Go version is required under Actions
diff --git a/go/extractor/cli/go-autobuilder/go-autobuilder.go b/go/extractor/cli/go-autobuilder/go-autobuilder.go
@@ -800,6 +800,9 @@ func installDependenciesAndBuild() {
 	// Go tooling should install required Go versions as needed.
 	if semver.Compare(getEnvGoSemVer(), "v1.21.0") < 0 && goVersionInfo.Found && semver.Compare("v"+goVersionInfo.Version, getEnvGoSemVer()) > 0 {
 		diagnostics.EmitNewerGoVersionNeeded()
+		if val, _ := os.LookupEnv("GITHUB_ACTIONS"); val == "true" {
+			log.Printf("The go.mod version is newer than the installed version of Go. Consider adding an actions/setup-go step to your workflow.\n")
+		}
 	}
 
 	fixGoVendorIssues(&buildInfo, goVersionInfo.Found)

Original file line number	Diff line number	Diff line change
`@@ -800,6 +800,9 @@ func installDependenciesAndBuild() {`
`800`	`800`	`// Go tooling should install required Go versions as needed.`
`801`	`801`	`if semver.Compare(getEnvGoSemVer(), "v1.21.0") < 0 && goVersionInfo.Found && semver.Compare("v"+goVersionInfo.Version, getEnvGoSemVer()) > 0 {`
`802`	`802`	`diagnostics.EmitNewerGoVersionNeeded()`
	`803`	`+ if val, _ := os.LookupEnv("GITHUB_ACTIONS"); val == "true" {`
	`804`	`+ log.Printf("The go.mod version is newer than the installed version of Go. Consider adding an actions/setup-go step to your workflow.\n")`
	`805`	`+ }`
`803`	`806`	`}`
`804`	`807`
`805`	`808`	`fixGoVendorIssues(&buildInfo, goVersionInfo.Found)`