feat: added news section, small updates#76
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (2)
📝 WalkthroughWalkthroughAdjusted catalog script logging range formatting, updated image reference CSS with centered block display and a new rounded-image class, and added a dated News section plus minor layout/link tweaks in documentation announcing automatic renderings. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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.
Actionable comments posted: 3
🧹 Nitpick comments (1)
source/_static/custom.css (1)
2-3:text-align: centerwill not center this block imageOn Line 3,
text-alignonimgis ineffective for centering the image itself after Line 2 setsdisplay: block. Use auto horizontal margins on the image (or applytext-alignon the parent).Suggested CSS adjustment
.image-reference img { display: block; - text-align: center; + margin-left: auto; + margin-right: auto; }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@source/_static/custom.css` around lines 2 - 3, The img rule currently sets display: block and text-align: center, but text-align on the img itself does nothing; change the centering approach for the image selector (the img rule) by replacing the ineffective text-align with auto horizontal margins (margin-left and margin-right set to auto) to center the block image, or alternatively remove text-align from the img rule and apply text-align: center to the parent container selector instead to achieve centering.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@scripts/generate-catalog.py`:
- Line 124: The logged upper bound end_log is computed incorrectly for
exclusive-end ranges: change the test-mode assignment of end_log (and the later
computation that sets end_log when end == total_count) to reflect that
range(offset, end) excludes end by using end_log = end - 1 (or end_log =
max(offset, end - 1)) instead of end_log = end or end_log = 1; update both
occurrences that set end_log (the test-mode branch and the runtime branch that
currently checks end == total_count) so the log that prints "offset-end_log"
shows the correct inclusive upper index when using range(offset, end).
In `@source/index.md`:
- Line 28: The News subsection heading currently uses "#### 2026-04-13" which
jumps the hierarchy from "##" to "####"; update that heading to "### 2026-04-13"
so the heading level remains consistent with the surrounding "##" News section
(replace the line containing "#### 2026-04-13" with "### 2026-04-13").
- Line 119: Replace the generic link text "here" on the line that links to
docs/snakemake with a descriptive phrase such as "Snakemake workflow authoring
guide" (or "Snakemake workflow examples"), so the sentence reads e.g. "Resources
for creating new workflows can be found in the Snakemake workflow authoring
guide (docs/snakemake) or in more detail on the Snakemake documentation
(https://snakemake.readthedocs.io/en/stable/index.html)"; update the visible
link text pointing to docs/snakemake accordingly.
---
Nitpick comments:
In `@source/_static/custom.css`:
- Around line 2-3: The img rule currently sets display: block and text-align:
center, but text-align on the img itself does nothing; change the centering
approach for the image selector (the img rule) by replacing the ineffective
text-align with auto horizontal margins (margin-left and margin-right set to
auto) to center the block image, or alternatively remove text-align from the img
rule and apply text-align: center to the parent container selector instead to
achieve centering.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 92e55f31-5dfa-4584-ba6e-f188aa71714e
⛔ Files ignored due to path filters (2)
source/_static/news_item1.pngis excluded by!**/*.pngsource/_static/news_item2.pngis excluded by!**/*.png
📒 Files selected for processing (3)
scripts/generate-catalog.pysource/_static/custom.csssource/index.md
Summary by CodeRabbit
New Features
Style
Documentation