Skip to content
This repository was archived by the owner on Jan 28, 2026. It is now read-only.

Conversation

@tschm
Copy link
Owner

@tschm tschm commented Sep 29, 2025

This PR updates configuration files from
tschm/.config-templates@main.

Summary by CodeRabbit

  • Documentation

    • Updated docs build to install required tooling automatically.
    • Exported Marimo notebooks now include code in the HTML output for better clarity.
    • Streamlined export process for more reliable documentation generation.
  • Chores

    • Adjusted CI triggers to run on both “main” and “master” branches, improving coverage for documentation workflows.

@coderabbitai
Copy link

coderabbitai bot commented Sep 29, 2025

Walkthrough

Expands the CI book workflow trigger to run on both main and master branches. Updates docs task to install marimo via uv and changes the marimo export command to use uv run with --include-code, removing --sandbox. Includes minor formatting updates in the docs task file.

Changes

Cohort / File(s) Summary
CI Workflow Triggers
\.github/workflows/book.yml
Broadened on.push.branches to include "main" and "master".
Docs Build Tasks
taskfiles/docs.yml
Added marimo installation via uv; switched export from uvx marimo export ... --sandbox to uv run marimo export ... --include-code; minor formatting tweaks.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as Developer
  participant Task as Task Runner (docs)
  participant UV as uv
  participant Marimo as marimo CLI

  Dev->>Task: Run docs task
  Task->>UV: pip install marimo
  UV-->>Task: marimo available
  Task->>UV: run marimo export html --include-code ...
  UV->>Marimo: invoke export
  Marimo-->>UV: HTML output
  UV-->>Task: Export complete
  Task-->>Dev: Docs built
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I thump my paws—two branches now align,
The presses hum, our book will build just fine.
I nibble docs with marimo delight,
Include the code, no sandbox in sight.
Hop, compile, and ship—so swift, so bright! 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title "chore: sync template from tschm/.config-templates@main" precisely describes the primary change of updating configuration files from the referenced template repository, using the conventional “chore” prefix for non-functional updates and avoiding unnecessary detail or vague wording.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch template-updates

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 92a3081 and ea5ee66.

📒 Files selected for processing (2)
  • .github/workflows/book.yml (1 hunks)
  • taskfiles/docs.yml (9 hunks)

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@tschm tschm merged commit 11a8935 into main Sep 29, 2025
13 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants