-
Notifications
You must be signed in to change notification settings - Fork 404
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
Forward Dokka Generator messages to Gradle logger #3833
base: master
Are you sure you want to change the base?
Forward Dokka Generator messages to Gradle logger #3833
Conversation
7d5cf5b
to
5bfc667
Compare
.build { | ||
output.invariantNewlines() shouldContain """ | ||
> Task :dokkaGenerateModuleHtml | ||
e: [:dokkaGenerateModuleHtml] test error message |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TBH, I don't think that duplicating task name will be useful, but I'm not against having it :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I put the task name in because it can be confusing if there are multiple generations at the same time, or if running with --quiet
. It's already a bit confusing when aggregating, because sometimes the same error is shown for both module and publication generation. But we can revisit the message if necessary.
Update Dokka Gradle tasks to forward DokkaGenerator logs to the appropriate levels.
KT-71719