Skip to content

go.mod: raise minimum go version to go1.24 and use tools directive#348

Open
thaJeztah wants to merge 3 commits intomicrosoft:mainfrom
thaJeztah:use_tools
Open

go.mod: raise minimum go version to go1.24 and use tools directive#348
thaJeztah wants to merge 3 commits intomicrosoft:mainfrom
thaJeztah:use_tools

Conversation

@thaJeztah
Copy link
Copy Markdown
Contributor

@thaJeztah thaJeztah commented Mar 23, 2026

go.mod: raise minimum go version to go1.24 and use tools directive

Update the minimum version to go1.24 so that we can use the tools
directive instead of using a "tools.go".

go.mod: update logrus to v1.9.4

Update the minimum version to drop some transitive dependencies.

@thaJeztah thaJeztah requested a review from a team as a code owner March 23, 2026 13:48
@thaJeztah
Copy link
Copy Markdown
Contributor Author

@AdityaMittal1306 @helsaawy PTAL

@thaJeztah
Copy link
Copy Markdown
Contributor Author

Ah, derp; linting issue will be fixed in my other PR;

Error: SA1019: runtime.GOROOT has been deprecated since Go 1.24: The root used during the Go build will not be meaningful if the binary is copied to another machine. Use the system path to locate the “go” binary, and use “go env GOROOT” to find its GOROOT.  (staticcheck)
  

```
tools/mkwinsyscall/mkwinsyscall.go:791:12: SA1019: runtime.GOROOT has been deprecated since Go 1.24: The root used during the Go build will not be meaningful if the binary is copied to another machine. Use the system path to locate
the “go” binary, and use “go env GOROOT” to find its GOROOT. (staticcheck)
	goroot := runtime.GOROOT()
	          ^
```

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Update the minimum version to go1.24 so that we can use the tools
directive instead of using a "tools.go".

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Update the minimum version to drop some transitive dependencies.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Copy Markdown
Contributor Author

@anmaxvl OK; I moved the commit over from the other branch, so linting should be happier now (but CI needs a kick to be started again 🤗)

@thaJeztah
Copy link
Copy Markdown
Contributor Author

cc @anmaxvl @jterry75 PTAL 🤗

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants