Skip to content

🔧 refactor: Replace interface{} with any in type definitions#1906

Merged
mkmccarty merged 1 commit intomainfrom
mm-branch-1
Dec 6, 2025
Merged

🔧 refactor: Replace interface{} with any in type definitions#1906
mkmccarty merged 1 commit intomainfrom
mm-branch-1

Conversation

@mkmccarty
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings December 6, 2025 04:18
@mkmccarty mkmccarty merged commit dfe9b2b into main Dec 6, 2025
8 checks passed
@mkmccarty mkmccarty deleted the mm-branch-1 branch December 6, 2025 04:18
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 the legacy interface{} type with the more idiomatic any type alias (introduced in Go 1.18) across the src/ei package. This refactoring improves code readability without changing functionality.

  • All map[string]interface{} type declarations replaced with map[string]any
  • Changes are consistent across test files and implementation files
  • No functional changes, purely a type alias update

Reviewed changes

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

File Description
src/ei/ei_units_test.go Updated test case struct field type and all test case option maps to use any instead of interface{}
src/ei/ei_units.go Updated FmtApprox function call to use map[string]any for consistency
src/ei/ei_research.go Updated multiple FormatEIValue function calls to use map[string]any for consistency

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.

1 participant