Skip to content

Update the README banner - #174

Merged
Jing-yilin merged 1 commit into
mainfrom
readme-banner
Sep 24, 2026
Merged

Jing-yilin merged 1 commit into
mainfrom
readme-banner

Conversation

@Jing-yilin

@Jing-yilin Jing-yilin commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Replaces assets/banner.webp with the new banner (neon wordmark, astronaut, "Agent harness for product design.", download and agent pills), rendered from the banner page's GitHub format at 1500×500 ×2 = 3000×1000.

🤖 Generated with Claude Code


Devin Review

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@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.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Devin Review found 2 potential issues.

⚠️ 2 issues in files not directly in the diff

⚠️ Welcome banner gains a black stripe

Regenerating the welcome board pads banner_uri() with 456 black pixels and cuts off the new banner's bottom. BANNER_CROP still targets the old 3456-pixel image, so the board displays a damaged banner.


⚠️ Welcome board keeps the old banner

The welcome board still embeds the old image after banner.webp changes. Its committed HTML stores a generated JPEG, so canvas visitors never see the replacement.

Devin Review

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying super-prototyping with  Cloudflare Pages  Cloudflare Pages

Latest commit: cc176ac
Status: ✅  Deploy successful!
Preview URL: https://804fdc77.super-prototyping.pages.dev
Branch Preview URL: https://readme-banner.super-prototyping.pages.dev

View logs

@Jing-yilin
Jing-yilin merged commit 7266e25 into main Sep 24, 2026
11 checks passed
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