Skip to content

fix(dokka): proper multi-module Dokka using root-level aggregation#412

Merged
wax911 merged 1 commit into
developfrom
fix/dokka-multimodule-proper
Jun 19, 2026
Merged

fix(dokka): proper multi-module Dokka using root-level aggregation#412
wax911 merged 1 commit into
developfrom
fix/dokka-multimodule-proper

Conversation

@wax911

@wax911 wax911 commented Jun 19, 2026

Copy link
Copy Markdown
Member

Replaces the per-module-task + hand-rolled-index.html approach with proper Dokka V2 multi-module aggregation as used in support-query-builder.

What changed:

  • Root build.gradle.kts: id("org.jetbrains.dokka") without version (resolved from buildSrc classpath)
  • dokka() dependencies aggregate all 8 subprojects into single output at dokka-docs/
  • Dokka natively produces per-module subdirectories — no manual index.html
  • CI simplified to single ./gradlew dokkaGeneratePublicationHtml
  • Added permissions: contents: write for deploy action

Verified:

  • ./gradlew dokkaGeneratePublicationHtml → BUILD SUCCESSFUL
  • Output at dokka-docs/ with 10 subdirectories (8 modules + images + scripts)
  • ./gradlew :app:compileDebugKotlin → BUILD SUCCESSFUL

Replace per-module task + hand-rolled index.html approach with proper
Dokka V2 multi-module aggregation as used in support-query-builder.

- Use id("org.jetbrains.dokka") without version at root (resolved from buildSrc)
- dokka() dependencies aggregate all 8 subprojects into single output
- Output at dokka-docs/ (Dokka natively produces per-module subdirectories)
- CI simplified to single ./gradlew dokkaGeneratePublicationHtml
- Added permissions: contents: write for deploy action
- Added dokka-docs/ to .gitignore
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto approved automated PR

@wax911 wax911 merged commit 946738d into develop Jun 19, 2026
11 checks passed
@wax911 wax911 deleted the fix/dokka-multimodule-proper branch June 19, 2026 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant