Skip to content

Fix the broken logo URL in the READMEs - #3

Merged
setoelkahfi merged 1 commit into
mainfrom
fix-readme-logo
Nov 2, 2026
Merged

setoelkahfi merged 1 commit into
mainfrom
fix-readme-logo

Conversation

@setoelkahfi

Copy link
Copy Markdown

The READMEs load the logo from ondeinference/onde on the development branch, which returns 404 (the file only exists on main). This points both the root README and the onde-code-agent crate README at main. The new URL returns 200.

The logo only exists on main in ondeinference/onde; the development branch URL returns 404. Fixes both the root README and the onde-code-agent README.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@sigit-code-review sigit-code-review 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.

Both READMEs now load the logo from refs/heads/main instead of refs/heads/development, fixing the 404 described in the PR. The two files change only that one URL each; nothing else is touched. The stated verification (new URL returns 200, old one 404) matches the diff, and there is nothing further to flag — the only residual risk is that the logo is pinned to a branch rather than a commit SHA, so it would break again if the asset moves off main, but that is pre-existing behavior and not introduced here.


Automated review by siGit Code Review · commit cd0d1c3 · see the review dashboard

@setoelkahfi setoelkahfi self-assigned this Nov 2, 2026
@setoelkahfi
setoelkahfi merged commit d5dbb41 into main Nov 2, 2026
5 checks passed
@foodandtravel
foodandtravel deleted the fix-readme-logo branch November 2, 2026 11:12
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