Skip to content

Conversation

@rrama
Copy link
Contributor

@rrama rrama commented Sep 8, 2025

What this does

With the new tool approach in go.mod, tools.go was no longer needed and has been removed.
The tool will automatically download when go tool is used if you do not already have the version downloaded.

Why the change?

No more will you have to remember to make clean and make install-tools, etc. when the version of the tool changes. No more will people run with the wrong version of the tool, causing inconsistencies with mocks in the codebase because someone forgot to do this.
Now you can go generate a single file without having to set up your PATH correctly first. This means you can click the green play button in your IDE to run the generate command in the file.

Notes for the reviewer

For more information on go tool see https://tip.golang.org/doc/go1.24#tools

Checklist

  • Tests added and all succeed
  • Linted
  • README.md updated, if user-facing

🚨After having merged, please update the snyk-ls and CLI go.mod to pull in latest client.

With the new tool approach in `go.mod`, `tools.go` was no longer needed and has been removed.
@snyk-io
Copy link

snyk-io bot commented Sep 8, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

code/snyk check is complete. No issues have been found. (View Details)

@rrama rrama marked this pull request as ready for review September 8, 2025 10:52
@rrama rrama requested review from a team as code owners September 8, 2025 10:52
@bastiandoetsch bastiandoetsch merged commit f69203f into main Sep 8, 2025
13 checks passed
@bastiandoetsch bastiandoetsch deleted the chore/IDE-1377_go-tool-command branch September 8, 2025 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants