docs: link the compiled paper from the book nav - #795
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe MkDocs navigation now includes a ChangesDocumentation navigation
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change adds the compiled paper PDF to the published book navigation without altering runtime behavior; no actionable merge-blocking risk remains after normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🟡 Changes recommended
The new nav entry points to a PDF that is gitignored and not present by default, which can break mkdocs build/serve unless the docs build reliably generates docs/paper/main.pdf beforehand.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds a MkDocs navigation entry to expose the compiled LaTeX paper PDF from the published documentation site, aligning with the existing docs/paper/README.md guidance and making the PDF discoverable via the book UI rather than only as a workflow artifact.
Changes:
- Add
Paper: paper/main.pdfto the MkDocsnavto link the compiled paper from the site navigation.
File summaries
| File | Description |
|---|---|
| mkdocs.yml | Adds a “Paper” nav entry pointing at the compiled PDF path (paper/main.pdf). |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
|
||
| nav: | ||
| - Home: index.md | ||
| - Paper: paper/main.pdf |
Adds one line to
mkdocs.yml:docs/paper/already sits insidedocs_dir, so the site build was copying the compiledPDF to
paper/main.pdfall along — it just wasn't linked from anywhere. This is the naventry the
paperbundle's own README recommends, and it turns the PDF from areachable-if-you-know-the-URL asset into the durable published copy (the workflow artifact
expires after 30 days).
Verified locally: compiled the paper, built the book, and ran the real gate —
rhiza-task book-navresolvespaper/main.pdfagainst_book/paper/main.pdf.🤖 Generated with Claude Code
Summary by CodeRabbit