Skip to content

Conversation

@qmuntal
Copy link
Member

@qmuntal qmuntal commented Jan 30, 2026

interface{} is a relic form the past. Rewrite it to any using gofmt -w -r 'interface{} -> any' ..

@qmuntal qmuntal requested a review from a team as a code owner January 30, 2026 15:07
Copilot AI review requested due to automatic review settings January 30, 2026 15:07
Copy link
Contributor

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

This PR modernizes the Go codebase by replacing all instances of the legacy interface{} type with the modern any alias, which was introduced in Go 1.18. The changes were made using gofmt -w -r 'interface{} -> any' ., ensuring consistency across the codebase.

Changes:

  • Replaced all occurrences of interface{} with any in type declarations, function signatures, and variable declarations
  • Updated code comments to reflect the new type nomenclature
  • Cleaned up trailing whitespace in README.md

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
go/types.go Updated struct field types and type aliases from interface{} to any
go/session.go Updated function parameter and variable types from interface{} to any
go/jsonrpc.go Updated JSON-RPC struct types and function signatures from interface{} to any
go/generated_session_events.go Updated generated code struct fields and utility function signatures from interface{} to any
go/e2e/session_test.go Updated test code type declarations from interface{} to any
go/definetool_test.go Updated test assertions and type casts from interface{} to any
go/definetool.go Updated function return types and comments from interface{} to any
go/client_test.go Updated test parameter types from interface{} to any
go/client.go Updated function signatures, variable declarations, and map types from interface{} to any
go/README.md Updated code examples from interface{} to any and removed trailing whitespace

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@friggeri
Copy link
Collaborator

@qmuntal I merged your other PRs, this one has a merge conflict

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.

2 participants