Skip to content

docs: add sponsor logos to readme - #212

Merged
jdx merged 1 commit into
mainfrom
chore/add-sponsor-logos
Aug 25, 2026
Merged

docs: add sponsor logos to readme#212
jdx merged 1 commit into
mainfrom
chore/add-sponsor-logos

Conversation

@jdx

@jdx jdx commented Aug 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace the text-only sponsor line with linked Entire and Omarchy wordmarks
  • select dark- or light-background assets based on the reader theme
  • retain the link to the complete sponsor list

AI-assisted — Tool: Codex; model: openai/gpt-5; version: unavailable.


Note

Low Risk
README-only documentation with external image URLs; no runtime or library behavior changes.

Overview
The Sponsors section in README.md no longer uses a plain text line naming Entire and Omacom Foundation. It now shows centered, linked wordmarks for both sponsors, using <picture> so dark vs light GitHub themes pick the right SVG assets from jdx.dev.

The View all sponsors link to jdx.dev/sponsors.html is unchanged; only presentation of the primary sponsors was upgraded.

Reviewed by Cursor Bugbot for commit f67e1bf. Bugbot is set up for automated code reviews on this repo. Configure here.

Summary by CodeRabbit

  • Documentation
    • Updated the Sponsors section with centered branding for Entire and the Omacom Foundation.
    • Added light and dark logo variants for improved theme support.
    • Added a “View all sponsors” link and removed the previous sponsorship text.

@jdx
jdx enabled auto-merge (squash) August 25, 2026 20:41
@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 6e8edbb8-395d-48f0-b58b-c5bd34e7bdba

📥 Commits

Reviewing files that changed from the base of the PR and between 63d70dd and f67e1bf.

📒 Files selected for processing (1)
  • README.md

Included review availability: Your plan provides up to 2 included reviews per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The README Sponsors section now uses centered HTML with responsive light/dark logos for Entire and the Omacom Foundation. It also includes links to both sponsors and a “View all sponsors” link.

Changes

Sponsor branding

Layer / File(s) Summary
README sponsor section
README.md
The Sponsors section replaces the Markdown sponsor sentence and standalone link with centered HTML, responsive sponsor logos, destination links, and a linked “View all sponsors” entry.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to f67e1

This is a localized README presentation change that adds linked sponsor logos without changing product behavior; no actionable merge-blocking risk remains.

Poem

A rabbit finds two logos bright

Light and dark, they greet the night
Centered links now neatly glow
Sponsors hop in tidy rows
“View all sponsors,” says the hare
A polished README waits there

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding sponsor logos to the README.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)


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.

@greptile-apps

greptile-apps Bot commented Aug 25, 2026

Copy link
Copy Markdown

Greptile Summary

This PR replaces the README’s text-only sponsor attribution with linked, theme-aware sponsor wordmarks while retaining the complete sponsor-list link.

  • Adds dark- and light-theme assets for Entire and Omacom Foundation.
  • Centers the sponsor presentation and preserves accessible image labels.
  • Leaves application code and runtime behavior unchanged.

Confidence Score: 5/5

The documentation-only change appears safe to merge.

The sponsor links remain intact, both logos provide fallback image sources and alternative text, and the complete sponsor-list link is preserved.

Important Files Changed

Filename Overview
README.md Replaces sponsor text links with centered, linked <picture> elements that select theme-specific SVG wordmarks; no actionable defect was identified.

Reviews (1): Last reviewed commit: "docs: add sponsor logos to readme" | Re-trigger Greptile

@jdx
jdx merged commit 3cd1dc2 into main Aug 25, 2026
10 checks passed
@jdx
jdx deleted the chore/add-sponsor-logos branch August 25, 2026 22:52
@jdx jdx mentioned this pull request Aug 25, 2026
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