update logo#38
Conversation
Signed-off-by: kerthcet <kerthcet@gmail.com>
|
/lgtm |
There was a problem hiding this comment.
Pull request overview
Updates the project branding by adding new theme-specific logo assets and wiring them into the README so the correct logo displays in light/dark mode.
Changes:
- Add new
logo-light.svgandlogo-dark.svgassets underdocs/images/. - Update the README header to use
<picture>sources for light/dark logos. - Minor README edits to the “Build from Source” and example output sections.
Reviewed changes
Copilot reviewed 1 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| docs/images/logo.png | Logo asset updated (binary change). |
| docs/images/logo-light.svg | Adds the new light-theme SVG logo. |
| docs/images/logo-dark.svg | Adds the new dark-theme SVG logo. |
| README.md | Switches README branding to theme-aware logos and adjusts a few documentation lines. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <img alt="PUMA Logo" src="docs/images/puma-logo-light.svg" width="240"> | ||
| <source media="(prefers-color-scheme: dark)" srcset="docs/images/logo-dark.svg"> | ||
| <source media="(prefers-color-scheme: light)" srcset="docs/images/logo-light.svg"> | ||
| <img alt="PUMA Logo" src="https://github.com/InftyAI/PUMA" width="240"> |
There was a problem hiding this comment.
The fallback in the
to an actual image asset (e.g., the light SVG in docs/images or a raw image URL).
| <img alt="PUMA Logo" src="https://github.com/InftyAI/PUMA" width="240"> | |
| <img alt="PUMA Logo" src="docs/images/logo-light.svg" width="240"> |
| @@ -42,7 +40,7 @@ cd PUMA | |||
| make build | |||
There was a problem hiding this comment.
The Build from Source section no longer lists prerequisites, but the commands below require at least Cargo/Rust and GNU Make (since it instructs running make build). Consider re-adding a short prerequisites note so new contributors don’t get stuck.
What this PR does / why we need it
Which issue(s) this PR fixes
Fixes #
Special notes for your reviewer
Does this PR introduce a user-facing change?