Skip to content

docs: improve README with original project logo#25

Merged
Th0rgal merged 3 commits into
mainfrom
docs/vector-logo-readme
Jul 27, 2026
Merged

docs: improve README with original project logo#25
Th0rgal merged 3 commits into
mainfrom
docs/vector-logo-readme

Conversation

@Th0rgal

@Th0rgal Th0rgal commented Jul 26, 2026

Copy link
Copy Markdown
Member

Summary

  • displays the original supplied PNG logo with its white background at the top of the README
  • avoids a redundant textual project heading because the project name is already part of the artwork
  • improves the README opening hierarchy while retaining the explicit verification-status caveat
  • replaces the macOS-specific “Mac owns” wording with the architecture-accurate “host owns” wording

Asset

  • original PNG, unchanged
  • RGB, 1254 × 1254
  • intrinsic white background
  • SHA-256: b3527350d4e4067009b2c1426072ddb3dbf34196aa0325ed2c030e0177888ba4

Validation

  • make checksum-check
  • make consistency
  • image format, mode and dimensions checked
  • README references the PNG and contains no redundant <h1> project name
  • git diff --check

@cursor

cursor Bot commented Jul 26, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_d0045a94-ce9a-4336-9e06-af7adf8ef043)

@cursor

cursor Bot commented Jul 26, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_27f70f71-9827-4d36-b1e6-27427ac2967b)

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2f60ca6445

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread SHA256SUMS Outdated
ca1b6b510bfeb5dc906aa0862e95d962c2901c246fa10fa72296d1666239d601 ./MANIFEST.md
c4e56018f86fd8b6fd1cd2e2cb39370b0501e8f83a1d7aeceab163457f9d60e4 ./Makefile
76702919f6a68922f243e5b2e59403d31ea74d8d33775dbb6411507e55cda2eb ./README.md
88ce833045477d93b9a0baca87463fd5b7801548a8ad0244866e1f40d41f1773 ./README.md

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Include the SVG in the checksum inventory

This refreshes the README checksum but omits the newly tracked docs/assets/lean-silicon-logo.svg. Consequently, tools/generate_checksums.py --check reports SHA256SUMS is stale, and the make check invocation in .github/workflows/ci.yml line 21 fails on every checkout of this commit. Regenerate the complete inventory after staging the SVG.

Useful? React with 👍 / 👎.

@cursor

cursor Bot commented Jul 26, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_2cfe4e99-7733-4587-a420-9ba54378fe87)

@Th0rgal Th0rgal changed the title docs: add vector leanSilicon logo to README docs: improve README with original project logo Jul 26, 2026

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b060fb418f

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md
Comment on lines +1 to +2
<p align="center">
<img src="docs/assets/lean-silicon-logo.png" alt="Project logo" width="520">

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Restore an accessible project heading

For screen-reader and text-only users, this replaces the only # leanSilicon heading with an image whose generic alt text says only “Project logo,” so the project name is absent from both the accessible content and the document heading outline. Keep a textual H1 (and ideally describe the wordmark in the alt text) rather than relying solely on the rasterized title.

Useful? React with 👍 / 👎.

@Th0rgal
Th0rgal merged commit 4dac896 into main Jul 27, 2026
9 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