Mention all-modules-page-plugin in README.md #62
Conversation
I understand this is required at the moment See adamko-dev#14
|
thanks 🙏 this would help |
README.md
Outdated
| @@ -145,6 +145,9 @@ dependencies { | |||
| // the subprojects must also have Dokkatoo applied | |||
| dokkatoo(projects(":subproject-hello")) | |||
| dokkatoo(projects(":subproject-world")) | |||
|
|
|||
| // This is required at the moment, see https://github.com/adamko-dev/dokkatoo/issues/14 | |||
| dokkatooPluginHtml("org.jetbrains.dokka:all-modules-page-plugin:1.7.20") | |||
There was a problem hiding this comment.
The version of Dokka that Dokkatoo uses by default is in a Property, which should be used so the version of Dokka is aligned.
| dokkatooPluginHtml("org.jetbrains.dokka:all-modules-page-plugin:1.7.20") | |
| dokkatooPluginHtml( | |
| dokkatoo.versions.jetbrainsDokka.map { dokkaVersion -> | |
| "org.jetbrains.dokka:all-modules-page-plugin:$dokkaVersion" | |
| } | |
| ) |
But I haven't checked this - would you be able to?
There was a problem hiding this comment.
Mmm, doesn't seem to work. Do you know where this is defined?
There was a problem hiding this comment.
Alright, I think this works?
dokkatooPluginHtml("org.jetbrains.dokka:all-modules-page-plugin:${org.jetbrains.dokka.DokkaVersion.version}")There was a problem hiding this comment.
What about it doesn't work? Does the script compile, or do the Gradle tasks run but it doesn't produce an aggregated publication?
dokkatoo is the regular DokkatooExtension Gradle extension, versions is an extension added to DokkatooExtension, and jetbrainsDokka is a regular Property<String> on DokkatooExtension.Versions
There was a problem hiding this comment.
What about it doesn't work?
Problem between chair and computer 😅 . I forgot part of your snippet and was expecting a top level dokkaVersion property 🤦 . I'll update
There was a problem hiding this comment.
It's pretty hardcore to use without generated accessors though.
There was a problem hiding this comment.
I've got a fix in the works for making it easier to use without accessors!
It might be easier just to (finally) fix the bug, so the workaround isn't necessary! I've got an idea, I'll give it a go.
I understand this is required for aggregation at the moment
See #14