Skip to content

[CODE-QUALITY] Docker image update checking is commented out #21

@MacJediWizard

Description

@MacJediWizard

Description

Image update checking functionality is commented out with a TODO note. This feature appears incomplete.

Location

internal/integrations/docker/docker.go lines 157-165

Code Snippet

// TODO: Make this configurable or run in background
// Check for updates (but don't block data collection)
// updates, updateErr := d.checkForImageUpdates(ctx, images)
// if updateErr != nil {
//     d.logger.WithError(updateErr).Warn("Failed to check for image updates")
// }

Impact

Docker image update checking is not functioning.

Recommended Fix

Either:

  1. Implement as a background task
  2. Make it configurable via settings
  3. Remove if no longer needed

Found By

Code quality audit

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions