Skip to content

update logo#38

Merged
InftyAI-Agent merged 1 commit into
InftyAI:mainfrom
kerthcet:cleanup/fix-model-name
Apr 25, 2026
Merged

update logo#38
InftyAI-Agent merged 1 commit into
InftyAI:mainfrom
kerthcet:cleanup/fix-model-name

Conversation

@kerthcet
Copy link
Copy Markdown
Member

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?


Signed-off-by: kerthcet <kerthcet@gmail.com>
Copilot AI review requested due to automatic review settings April 25, 2026 18:12
@InftyAI-Agent InftyAI-Agent added needs-triage Indicates an issue or PR lacks a label and requires one. needs-priority Indicates a PR lacks a label and requires one. do-not-merge/needs-kind Indicates a PR lacks a label and requires one. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Apr 25, 2026
@kerthcet
Copy link
Copy Markdown
Member Author

/lgtm
/kind documentation

@InftyAI-Agent InftyAI-Agent added lgtm Looks good to me, indicates that a PR is ready to be merged. documentation Categorizes issue or PR as related to documentation. and removed do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Apr 25, 2026
@InftyAI-Agent InftyAI-Agent merged commit 7fa15f5 into InftyAI:main Apr 25, 2026
42 of 44 checks passed
@kerthcet kerthcet deleted the cleanup/fix-model-name branch April 25, 2026 18:13
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.svg and logo-dark.svg assets under docs/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.

Comment thread README.md
<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">
Copy link

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

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

The fallback in the tag points to the GitHub repository URL (an HTML page), so the logo will not render (and the fallback will be broken in clients that ignore ). Point the to an actual image asset (e.g., the light SVG in docs/images or a raw image URL).

Suggested change
<img alt="PUMA Logo" src="https://github.com/InftyAI/PUMA" width="240">
<img alt="PUMA Logo" src="docs/images/logo-light.svg" width="240">

Copilot uses AI. Check for mistakes.
Comment thread README.md
Comment on lines 32 to 40
@@ -42,7 +40,7 @@ cd PUMA
make build
Copy link

Copilot AI Apr 25, 2026

Choose a reason for hiding this comment

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

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.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. documentation Categorizes issue or PR as related to documentation. lgtm Looks good to me, indicates that a PR is ready to be merged. needs-priority Indicates a PR lacks a label and requires one. needs-triage Indicates an issue or PR lacks a label and requires one.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants