Skip to content

Validate logger migration completeness across targeted packages#34122

Closed
Copilot wants to merge 1 commit into
mainfrom
copilot/deep-report-migrate-log-calls
Closed

Validate logger migration completeness across targeted packages#34122
Copilot wants to merge 1 commit into
mainfrom
copilot/deep-report-migrate-log-calls

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 22, 2026

DeepReport reported residual log.Print* usage in several library packages, requiring migration to pkg/logger. This PR verifies the current tree state for those paths and confirms the migration is already complete with no remaining production-library violations.

  • Issue scope verified

    • Re-checked the reported package set (fileutil, gitutil, semverutil, typeutil, repoutil, envutil, workflow, cli) for stdlib log.Print* usage.
    • Confirmed these packages are using namespaced pkg/logger instances.
  • Code delta

    • No source changes were required; the repository already satisfies the logger-migration requirement for production library code.
    • Remaining log.Print* occurrences are confined to logger docs/tests and linter test fixtures, which are expected.
  • Representative pattern in current code

    import "github.com/github/gh-aw/pkg/logger"
    
    var semverLog = logger.New("semverutil:semverutil")
    
    func ParseVersion(v string) *SemanticVersion {
        semverLog.Printf("Parsing semantic version: %s", v)
        // ...
    }

Copilot AI changed the title [WIP] Migrate 38 log.Print calls to pkg/logger across 8 packages Validate logger migration completeness across targeted packages May 22, 2026
Copilot AI requested a review from pelikhan May 22, 2026 22:31
@pelikhan pelikhan closed this May 22, 2026
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.

[deep-report] Migrate 38 log.Print* calls to pkg/logger across 8 packages

2 participants