Skip to content

Conversation

jakebailey
Copy link
Member

@jakebailey jakebailey commented Sep 9, 2025

  • Update to gofumpt v0.9
    • This now formats to fix naked returns.
    • I removed the -lang=... bit from everywhere, which I don't think does anything anymore. Was needed at some point in the past.
    • dprint now lets you configure a filename for the cache key, so use go.mod.
    • Downside: gopls still vendors v0.8. Oh well.
  • Update the rest of the deps, and update NOTICE.

@Copilot Copilot AI review requested due to automatic review settings September 9, 2025 17:53
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates dependencies to their latest versions (particularly gofumpt to v0.9) and removes the deprecated -lang=... parameter from gofumpt commands throughout the codebase. The new gofumpt version automatically formats code to fix naked returns, making the codebase more consistent with Go best practices.

  • Updated all dependencies in go.mod and _tools/go.mod to their latest versions
  • Removed -lang=... parameter from all gofumpt commands in go:generate directives and configuration files
  • Fixed naked returns throughout the codebase by explicitly returning named values

Reviewed Changes

Copilot reviewed 23 out of 25 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
go.mod Updated main module dependencies to latest versions
_tools/go.mod Updated tools dependencies to latest versions
NOTICE.txt Updated dependency license information to reflect new versions
.dprint.jsonc Removed -lang=go1.25 parameter from gofumpt command
Multiple internal/*.go files Removed -lang=go1.25 parameter from go:generate directives
Multiple internal/*.go files Fixed naked returns by explicitly returning values

@jakebailey jakebailey added this pull request to the merge queue Sep 9, 2025
Merged via the queue into main with commit 1ebf658 Sep 9, 2025
22 checks passed
@jakebailey jakebailey deleted the jabaile/update-deps-2025-09-02 branch September 9, 2025 18:44
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