Skip to content

fixed the linter warnings#40

Merged
kommendorkapten merged 1 commit intomainfrom
linterfix
Feb 17, 2026
Merged

fixed the linter warnings#40
kommendorkapten merged 1 commit intomainfrom
linterfix

Conversation

@kommendorkapten
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings February 17, 2026 15:52
@kommendorkapten kommendorkapten requested a review from a team as a code owner February 17, 2026 15:52
Copy link

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 addresses lint issues by aligning Go package names with their directory/import paths, and updating call sites accordingly.

Changes:

  • Renamed package declarations to match their directories (pkg/ociutilpackage ociutil, pkg/dtmetricspackage dtmetrics).
  • Updated imports/usages in the controller and deployment record client to use ociutil and dtmetrics.
  • Added gosec suppression comments for credential-like config fields / HTTP request call sites.

Reviewed changes

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

Show a summary per file
File Description
pkg/ociutil/name_test.go Update test package name to ociutil.
pkg/ociutil/name.go Update package name to ociutil.
pkg/ociutil/digest_test.go Update test package name to ociutil.
pkg/ociutil/digest.go Update package name to ociutil.
pkg/dtmetrics/prom.go Update package name to dtmetrics.
pkg/deploymentrecord/client.go Switch metrics import/usage to dtmetrics; add gosec suppression near HTTP call.
internal/controller/controller.go Switch imports/usages from metrics/image to dtmetrics/ociutil.
internal/controller/config.go Add gosec suppression for credential-like config fields and re-align struct fields.

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

@kommendorkapten kommendorkapten merged commit c33a488 into main Feb 17, 2026
12 checks passed
@kommendorkapten kommendorkapten deleted the linterfix branch February 17, 2026 15:59
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