Demote recompiling message from INFO to DEBUG (#333) - #1089
Conversation
There was a problem hiding this comment.
Pull request overview
Reduces default build output noise during incremental builds by lowering the verbosity of the “Recompiling all files because …” reason message so it no longer appears at the default INFO level for every module.
Changes:
- Changed the incremental full-rebuild “cause” log line from INFO to DEBUG in
ToolExecutor.applyIncrementalBuild(...).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ascheman
left a comment
There was a problem hiding this comment.
LGTM — demoting the per-module "Recompiling all files…" message to DEBUG is a sensible noise reduction for the default log level, and it's guarded by the existing !sourceFiles.isEmpty() check so it won't fire on empty sets. Thanks @elharo.
|
@elharo Please assign appropriate label to PR according to the type of change. |
|
The comment in the review saying that we should probably keep the I have create pull request #1108 for the |
I like this idea (we have discussed and applied it elsewhere already). |
Part of #333
Changes the 'Recompiling all files because ...' log message from INFO to DEBUG level. This message is emitted for every module during incremental builds and is noisy at the default log level.