Skip to content

When stderr is redirected to a file, don't spam Ignoring SetDefaultsForClientTools() unless loglevel is verbose#79

Merged
ldemailly merged 2 commits intomainfrom
noinfo_on_redirect
Nov 10, 2025
Merged

When stderr is redirected to a file, don't spam Ignoring SetDefaultsForClientTools() unless loglevel is verbose#79
ldemailly merged 2 commits intomainfrom
noinfo_on_redirect

Conversation

@ldemailly
Copy link
Copy Markdown
Member

Avoids

{"ts":1762733319.925241,"level":"info","r":1,"file":"logger.go","line":162,"msg":"Ignoring SetDefaultsForClientTools() call due to non console logging"}

@ldemailly
Copy link
Copy Markdown
Member Author

@copilot can you make a PR off this to fix the godoc golint errors (which are unrelated to this yet blocking because the shared workflows got a newer set of stricter linters)

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 10, 2025

@ldemailly I've opened a new pull request, #80, to work on those changes. Once the pull request is ready, I'll request review from you.

* Initial plan

* Fix godoc lint errors for stricter linters

Co-authored-by: ldemailly <3664595+ldemailly@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ldemailly <3664595+ldemailly@users.noreply.github.com>
@ldemailly ldemailly merged commit da8c50b into main Nov 10, 2025
6 checks passed
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.

2 participants