Conversation
Signed-off-by: kerthcet <kerthcet@gmail.com>
InftyAI-Agent
added
needs-triage
|
/lgtm |
InftyAI-Agent
added
lgtm
There was a problem hiding this comment.
Pull request overview
Updates the repository’s branding assets by introducing a new SVG logo and switching the README’s logo <picture> sources to use it.
Changes:
- Add
docs/images/logo.svg(new SVG logo asset). - Update
README.mdto referencedocs/images/logo.svgfor both dark/light<source>entries.
Reviewed changes
Copilot reviewed 1 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/images/logo.svg | Adds the new SVG logo artwork for documentation/README usage. |
| README.md | Switches the README logo <picture> sources from PNG to the new SVG. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <source media="(prefers-color-scheme: light)" srcset="docs/images/logo.png"> | ||
| <source media="(prefers-color-scheme: dark)" srcset="docs/images/logo.svg"> | ||
| <source media="(prefers-color-scheme: light)" srcset="docs/images/logo.svg"> | ||
| <img alt="PUMA Logo" src="docs/images/puma-logo-light.svg" width="240"> |
There was a problem hiding this comment.
<img> fallback points to docs/images/puma-logo-light.svg, but that file does not exist in docs/images/ (and there are no repo matches). This will render a broken image in contexts where the <source> elements aren’t used (or if SVG fails to load). Update the fallback src to an existing asset (e.g., the new docs/images/logo.svg or the existing docs/images/logo.png) and/or add the missing file.
| <img alt="PUMA Logo" src="docs/images/puma-logo-light.svg" width="240"> | |
| <img alt="PUMA Logo" src="docs/images/logo.svg" width="240"> |
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?