chore: Improve log output #553

Closed

wetneb wants to merge 2 commits from better-logging into main

wetneb commented

2025-07-30 13:02:41 +02:00

Owner

A lot of logging statements detailing the construction of the merged tree were very rarely useful in debugging. Switching them to the trace level means we'd still be able to activate them if needed, but they won't be shown anymore in verbose mode (as that mode is set to the debug level).
For the purposes of detecting which type of merge was returned (in benchmarks, #537) I have added a corresponding log line.

* A lot of logging statements detailing the construction of the merged tree were very rarely useful in debugging. Switching them to the trace level means we'd still be able to activate them if needed, but they won't be shown anymore in verbose mode (as that mode is set to the debug level). * For the purposes of detecting which type of merge was returned (in benchmarks, #537) I have added a corresponding log line.

wetneb added 2 commits

2025-07-30 13:02:42 +02:00

chore: Downgrade many log statements from debug to trace level f0fd35d44b

chore: Add logging statement to make it easier to detect which merge was returned

/ test (pull_request) Successful in 44s

Details

1e5fb45334

ada4a approved these changes

2025-08-02 14:59:37 +02:00

wetneb commented

2025-08-02 18:35:26 +02:00

Author

Owner

This was merged but there was a Codeberg glitch which left the PR open. So I'm closing it manually. (See forgejo/forgejo#3518)

This was merged but there was a Codeberg glitch which left the PR open. So I'm closing it manually. (See https://codeberg.org/forgejo/forgejo/issues/3518)