-
Notifications
You must be signed in to change notification settings - Fork 56
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
feat: added dokka templates directory for kotlinlang #377
base: master
Are you sure you want to change the base?
Conversation
3837955
to
20ce356
Compare
20ce356
to
9f4b123
Compare
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.
@berezinant thank you for working on docs publication!
There are a few minor changes that could slightly improve build scripts, but otherwise the change looks good.
@@ -44,3 +44,6 @@ kover { | |||
} | |||
} | |||
} | |||
tasks.dokkaHtmlMultiModule { | |||
pluginsMapConfiguration.set(mapOf("org.jetbrains.dokka.base.DokkaBase" to """{ "templatesDir" : "${projectDir.toString().replace('\\', '/')}/dokka-templates" }""")) |
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.
pluginsMapConfiguration.set(mapOf("org.jetbrains.dokka.base.DokkaBase" to """{ "templatesDir" : "${projectDir.toString().replace('\\', '/')}/dokka-templates" }""")) | |
pluginsMapConfiguration.set(mapOf("org.jetbrains.dokka.base.DokkaBase" to """{ "templatesDir" : "${projectDir.resolve("dokka-templates")}" }""")) |
Or there's a reason to manually replace backslashes?
@@ -44,3 +44,6 @@ kover { | |||
} | |||
} | |||
} | |||
tasks.dokkaHtmlMultiModule { |
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.
Could you please move this config setup to kotlinx-io-dokka.gradle.kts too?
I guess, dokkaHtmlMultiModule
would need to be replaced with withType(DokkaMultiModuleTask::class.java)
.
We're adding kotlinx-io api reference to kotlinlang and we need to customise templates for that