Skip to content

refactor: standardized logger parameter bind type-assured #1772

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

refactor: standardized logger parameter bind type-assured #1772

wants to merge 3 commits into from

Conversation

@loveysuby
Copy link
Contributor

@loveysuby loveysuby commented Nov 20, 2024

  • Improving memory occupancy of unnecessary log data by formatting log parameter targets in test logger data dummy.
  • The parsed string comes in from the previous row, but the log output specification is readable. :)
  • Run a build and make sure all tests pass prior to submission
    test-passed-snapshot-spring-ai-PR-standardized-logger

@ericbottard
Copy link
Member

ericbottard commented Oct 1, 2025

Applied on main via 17c8e9e

@ericbottard ericbottard closed this Oct 1, 2025
@loveysuby
Copy link
Contributor Author

loveysuby commented Oct 1, 2025 •
edited
Loading

Thanks for resolving. @ericbottard

@loveysuby loveysuby deleted the refactor-logger-format-consistency branch October 1, 2025 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ericbottard ericbottard ericbottard approved these changes

Assignees

@ericbottard ericbottard

Labels

code cleanup

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@loveysuby @ericbottard @markpollack