Skip to content

Fix DocC build: generate targets separately and merge outputs#68

Merged
albertodebortoli merged 1 commit intomainfrom
fix/docc-merge-outputs
Apr 12, 2026
Merged

Fix DocC build: generate targets separately and merge outputs#68
albertodebortoli merged 1 commit intomainfrom
fix/docc-merge-outputs

Conversation

@albertodebortoli
Copy link
Copy Markdown
Member

Summary

  • Fixes Move DocC tutorials from LucaCore to LucaCLI catalog #66: the second generate-documentation invocation was clearing the output directory before writing, which wiped the LucaCore docs and left only documentation/lucacli/ reachable
  • Each target is now built into its own temporary directory (./docs-lucacore and ./docs)
  • After both builds complete, LucaCore's target-specific subdirectories (documentation/lucacore/ and data/documentation/lucacore/) are copied into the LucaCLI output — no conflict since each target owns a unique path
  • The shared static assets (CSS, JS) come from the LucaCLI build; they are identical to what LucaCore would have produced

Test plan

  • Verify CI DocC workflow passes
  • Confirm https://lucatools.github.io/Luca/documentation/lucacli/ is reachable after merge
  • Confirm https://lucatools.github.io/Luca/documentation/lucacore/ is reachable after merge

@albertodebortoli albertodebortoli added this to the 0.15.0 milestone Apr 12, 2026
@albertodebortoli albertodebortoli added the documentation Improvements or additions to documentation label Apr 12, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@albertodebortoli albertodebortoli merged commit 5fa69bc into main Apr 12, 2026
3 checks passed
@albertodebortoli albertodebortoli deleted the fix/docc-merge-outputs branch April 12, 2026 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant