Skip to content
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.

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

Fix to use trace for verbose debug logs #92

Merged
merged 2 commits into from
Nov 20, 2023
Merged

Conversation

lovasoa
Copy link
Contributor

@lovasoa lovasoa commented Nov 19, 2023

moves the most verbose logs from DEBUG level to TRACE, to make logs more readable

fixes #91

moves the most verbose logs from DEBUG level to TRACE, to make logs more readable

Signed-off-by: Ophir LOJKINE <contact@ophir.dev>
@wooorm
Copy link
Owner

wooorm commented Nov 19, 2023

I think there's docs on how to debug, could you change those too?

Signed-off-by: Ophir LOJKINE <contact@ophir.dev>
@lovasoa
Copy link
Contributor Author

lovasoa commented Nov 19, 2023

I updated the README. Is there anything else you want me to change ?

@wooorm wooorm changed the title reduce log verbosity at debug level Fix to use trace for verbose debug logs Nov 20, 2023
@wooorm wooorm merged commit 8ea1401 into wooorm:main Nov 20, 2023
2 checks passed
@lovasoa lovasoa deleted the patch-1 branch November 20, 2023 10:50
@lovasoa
Copy link
Contributor Author

lovasoa commented Nov 20, 2023

great, thanks for merging ! Could you publish a new version, so that I can update SQLPage's dependencies ?

@wooorm
Copy link
Owner

wooorm commented Nov 20, 2023

working on it! :)

@wooorm
Copy link
Owner

wooorm commented Nov 20, 2023

released!

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.

log spamming
2 participants