Skip to content

docs: Add the README banner from master - #107

Merged
BenWestgate merged 1 commit into
reviewability-v1from
claude/readme-banner-v1
Oct 2, 2026
Merged

BenWestgate merged 1 commit into
reviewability-v1from
claude/readme-banner-v1

Conversation

@BenWestgate

Copy link
Copy Markdown
Owner

What

Cherry-picks 4d7cf29 ("add README.md banner image", #83) from master with -x. It adds docs/images/python-codex32-banner.jpg and puts the banner line at the top of the v1 README.

Why

The banner is on master but not reviewability-v1. Bringing it over now means #94 won't conflict over the README later.

Notes

  • This is a cherry-pick, not a merge. master has three other commits reviewability-v1 lacks (the 0.6.1 and 0.6.2 version bumps and efece7d). A merge would bring those in and would add a merge commit, which reviewability-v1's linear-history rule rejects.
  • The commit keeps Ben as author; its message gains the standard (cherry picked from commit …) line.
  • Land it by fast-forward to keep the hash.

Validation: the cherry-pick applied cleanly, and the diff is identical to 4d7cf29 (2 files, +2 lines). It changes no code.

🤖 Generated with Claude Code

https://claude.ai/code/session_013gZvwvuocM7a7Ut4kiBFHw


Generated by Claude Code

@BenWestgate BenWestgate self-assigned this Oct 2, 2026

@BenWestgate BenWestgate left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BenWestgate
BenWestgate marked this pull request as ready for review October 2, 2026 06:20
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@BenWestgate
BenWestgate enabled auto-merge (rebase) October 2, 2026 06:21
@BenWestgate
BenWestgate disabled auto-merge October 2, 2026 06:23
(cherry picked from commit 4d7cf29)
@BenWestgate
BenWestgate force-pushed the claude/readme-banner-v1 branch from 6956a4e to 6d29e65 Compare October 2, 2026 06:31
@BenWestgate
BenWestgate enabled auto-merge (rebase) October 2, 2026 06:32

Copy link
Copy Markdown
Owner Author

AI-written (Claude), disclosed per docs/developer/AI_POLICY.md.

The github-advanced-security check failure isn't caused by this PR. That job runs GitHub's Copilot autofind reviewer, and it stopped before analysing anything: SessionModelError: You have exceeded your monthly quota (HTTP 402). This PR changes only the README and adds an image, so there's no code fix to make. Re-running won't help until the Copilot quota resets or is raised. The CodeQL checks are separate.


Generated by Claude Code

@BenWestgate
BenWestgate merged commit 5325eb7 into reviewability-v1 Oct 2, 2026
23 of 24 checks passed
@BenWestgate
BenWestgate deleted the claude/readme-banner-v1 branch October 2, 2026 06:42
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