Skip to content

[AIR-3063] qs: use logger from voedger#141

Merged
host6 merged 2 commits intountillpro:mainfrom
host6:AIR-3063-qs-use-logger-from-voedger-pr
Feb 24, 2026
Merged

[AIR-3063] qs: use logger from voedger#141
host6 merged 2 commits intountillpro:mainfrom
host6:AIR-3063-qs-use-logger-from-voedger-pr

Conversation

@host6
Copy link
Collaborator

@host6 host6 commented Feb 24, 2026

@host6 host6 merged commit 0c1545a into untillpro:main Feb 24, 2026
0 of 3 checks passed
@host6 host6 deleted the AIR-3063-qs-use-logger-from-voedger-pr branch February 24, 2026 14:37
@augmentcode
Copy link

augmentcode bot commented Feb 24, 2026

🤖 Augment PR Summary

Summary: This PR migrates qs to use the logger implementation provided by the Voedger project.

Changes:

  • Replaced imports of github.com/untillpro/goutils/logger with github.com/voedger/voedger/pkg/goutils/logger across git commands, CLI command processing, and utility code
  • Updated the module dependencies to include Voedger so the new logger package can be referenced

Technical Notes: This is a package-level swap only (no call-site logic changes), so runtime behavior depends on Voedger logger defaults (format/levels/output).

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.


"github.com/untillpro/goutils/logger"
"github.com/untillpro/qs/internal/cmdproc"
"github.com/voedger/voedger/pkg/goutils/logger"
Copy link

Choose a reason for hiding this comment

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

This PR switches to github.com/voedger/voedger/pkg/goutils/logger, but .augment/rules/ar-coding-standards.md still mandates github.com/untillpro/goutils/logger. Consider updating that rule to avoid conflicting guidance for future changes/reviews.

Severity: low

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

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