Skip to content

Conversation

@alarso16
Copy link
Contributor

Why this should be merged

It's unnecessary to have multiple go.mods for tooling, since we can just update the legacy config files.

How this works

  1. Run go tool -modfile=tools/go.mod golangci-lint migrate. Copy comments over.
  2. Run the new linter and note all staticcheck failures. Disable them.
  3. Remove some unused code
  4. Delete the legacy tool go.mod

How this was tested

Existing CI

Need to be documented?

No

Need to update RELEASES.md?

No

@alarso16 alarso16 self-assigned this Oct 28, 2025
@alarso16 alarso16 added dependencies Pull requests that update a dependency file ci labels Oct 28, 2025
@alarso16 alarso16 marked this pull request as ready for review October 28, 2025 16:03
@alarso16 alarso16 requested review from a team and maru-ava as code owners October 28, 2025 16:03
@maru-ava maru-ava added this pull request to the merge queue Oct 29, 2025
Merged via the queue into master with commit a3a4755 Oct 29, 2025
9 checks passed
@maru-ava maru-ava deleted the alarso16/upgrade-legacy-linter branch October 29, 2025 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants