Add RDoc logo to documentation#1734
Conversation
|
🚀 Preview deployment available at: https://d250f31f.rdoc-6cd.pages.dev (commit: 0011d0a) |
There was a problem hiding this comment.
⚠️ Not ready to approve
The new README image reference will produce a broken image when users generate docs via the plain rdoc command unless .rdoc_options (or equivalent CLI guidance) is updated to copy the asset too.
Pull request overview
This PR adds an RDoc logo asset and wires it into both the repository README and generated Aliki-theme documentation output, ensuring the image is packaged with the gem and copied into generated site output.
Changes:
- Add
rdoc-logo.svgand reference it fromREADME.md. - Ensure the SVG is included in gem packaging and copied during RDoc generation (
rdoc.gemspec,Rakefile). - Add Aliki CSS rules to style the logo block in generated HTML.
File summaries
| File | Description |
|---|---|
README.md |
Adds a centered logo image at the top of the README. |
rdoc.gemspec |
Packages the SVG and adds --copy-files rdoc-logo.svg to gem RDoc generation options. |
rdoc-logo.svg |
Introduces the new logo asset as an SVG. |
Rakefile |
Adds --copy-files rdoc-logo.svg to the rake rdoc task options so site builds include the asset. |
lib/rdoc/generator/template/aliki/css/rdoc.css |
Styles the logo block for the generated docs (including a dark-theme variant). |
Copilot's findings
- Files reviewed: 4/5 changed files
- Comments generated: 1
Note
Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Ideally this should be under doc/. But moving it there would cause file path mismatch in generated doc (copied file is always placed on _site/, so source will still be rdoc-logo.svg) and GH (will use doc/rdoc-logo.svg). So I'll leave that to another discussion/PR.
On official website:
On GitHub: