Skip to content

Modernize codebase: replace interface{} with any - #595

Merged
gab-arrobo merged 2 commits into
omec-project:mainfrom
gab-arrobo:refactor-any
Oct 6, 2026
Merged

gab-arrobo merged 2 commits into
omec-project:mainfrom
gab-arrobo:refactor-any

Conversation

@gab-arrobo

Copy link
Copy Markdown
Contributor

Also update dependencies

Signed-off-by: Arrobo, Gabriel <gabriel.arrobo@intel.com>
@gab-arrobo
gab-arrobo requested review from a team and a balanced review from Copilot October 6, 2026 18:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

db_adapter.go is not gofmt-compliant and will fail the configured formatting checks.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Modernizes Go code by replacing interface{} with its any alias and updates dependencies.

Changes:

  • Replaced remaining interface{} usages with any.
  • Updated four Go dependencies and checksums.
  • Updated affected mocks and tests.
File Description
go.mod Updates dependency versions.
go.sum Refreshes dependency checksums.
dbadapter/​db_adapter.go Modernizes database API types.
configmodels/​model_slice_site_info.go Modernizes the UPF map type.
configapi/​slice_operations_test.go Updates type assertions.
configapi/​slice_helpers.go Modernizes document map types.
configapi/​common_mock_db.go Updates transaction mock types.
backend/​webui_context/​context.go Modernizes decoder types.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread dbadapter/db_adapter.go Outdated
Signed-off-by: Arrobo, Gabriel <gabriel.arrobo@intel.com>
Copilot AI balanced review requested due to automatic review settings October 6, 2026 18:38

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The alias migration is behavior-preserving and consistent, while dependency versions and checksums align.

Review effort: Balanced
Findings: None

Resolved since last review (1)

@gab-arrobo
gab-arrobo merged commit da341b7 into omec-project:main Oct 6, 2026
13 checks passed
@gab-arrobo
gab-arrobo deleted the refactor-any branch October 6, 2026 18:57
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