Skip to content

Update README.md - #450

Merged
gkoch02 merged 1 commit into
mainfrom
gkoch02-patch-1
Oct 16, 2026
Merged

gkoch02 merged 1 commit into
mainfrom
gkoch02-patch-1

Conversation

@gkoch02

@gkoch02 gkoch02 commented Oct 16, 2026

Copy link
Copy Markdown
Owner

Summary

Trade-offs

Testing

  • ruff check . passes
  • pytest passes locally
  • New or changed behaviour has a test that fails on main without the fix
  • Renderer change: golden fixtures either still pass, or were intentionally
    re-baselined with UPDATE_RENDER_GOLDEN=1 pytest tests/test_render_golden.py
    and the regenerated PNGs are in this PR
  • Corpus change: idle-hours bake was run and quote_database.jsonl is
    committed alongside the raw corpus

Appliance impact

Docs

  • The one doc that owns the fact updated (docs/pipeline.md / runtime.md /
    themes.md / web_ui.md / testing.md), and CLAUDE.md only if an
    invariant changed. Link rather than restate, and don't write registry
    counts ("N themes") into prose.
  • README.md updated if this changes a user-facing command or the theme table

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-16T21:40:24.366278Z de23610 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@gkoch02
gkoch02 enabled auto-merge October 16, 2026 21:42
@gkoch02
gkoch02 merged commit 428df42 into main Oct 16, 2026
13 checks passed
@gkoch02
gkoch02 deleted the gkoch02-patch-1 branch October 16, 2026 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant