Skip to content

fix(dts): unify logger in dts plugin#1438

Merged
Timeless0911 merged 1 commit intomainfrom
feat/unified-logger
Jan 8, 2026
Merged

fix(dts): unify logger in dts plugin#1438
Timeless0911 merged 1 commit intomainfrom
feat/unified-logger

Conversation

@Timeless0911
Copy link
Copy Markdown
Contributor

Summary

Updated packages/plugin-dts/src/tsgo.ts to use logger.log instead of console.log. This ensures that the log level configuration is respected and consistent with the rest of the application.

Related Links

N/A

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

## Summary
Updated `packages/plugin-dts/src/tsgo.ts` to use `logger.log` instead of `console.log`. This ensures that the log level configuration is respected and consistent with the rest of the application.

## Related Links
N/A

## Checklist
- [x] Tests updated (or not required).
- [x] Documentation updated (or not required).
Copilot AI review requested due to automatic review settings January 8, 2026 09:50
@Timeless0911 Timeless0911 changed the title fix: unify log calling convention in dts plugin fix(dts): unify logger in dts plugin Jan 8, 2026
Copy link
Copy Markdown
Contributor

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 standardizes logging in the dts plugin by replacing a console.log call with logger.log, ensuring consistency with the rest of the codebase and respecting log level configuration.

  • Replaced console.log with logger.log in the tsgo output handler

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

@Timeless0911 Timeless0911 merged commit 65f850e into main Jan 8, 2026
23 checks passed
@Timeless0911 Timeless0911 deleted the feat/unified-logger branch January 8, 2026 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants