Use info level for progress in Dokka Gradle plugin #2915
Conversation
| "progress" -> logger.lifecycle(message) | ||
| "progress" -> logger.info(message) |
There was a problem hiding this comment.
Gradle's lifecycle level (described as Progress information messages) is the ideal mapping option for Dokka's progress level, so I don't think this change is right, sorry
|
Unfortunately, the mentioned problem is not as easy to fix. I might be wrong, but it looks like the proposed solution would silence Dokka overall, no messages will be emitted other than warnings and errors, so for the users it'll look like Dokka is broken all of a sudden. A proper solution would require revisiting all messages that are logged by Dokka on all levels, and it's also not as easy to decide which messages need to be re-written, which need another level and so on. If you really want to help with it, it'll take time and we'll need to agree on the messages beforehand so that you don't waste time implementing something we wouldn't be able to accept. Either way, I'll close this PR as it's not really addressing the problem, and we can contibue the discussion on what needs to be done in the issue itself Thank you for wanting to contribute though, we appreciate it |
|
I haven't touch warnings and errors, also |
fixes #1894
Level for progress set to
info. See the issue for more details