Skip to content

Update README.md#24649

Merged
ishaan-berri merged 1 commit intomainfrom
ishaan-berri-patch-1
Mar 26, 2026
Merged

Update README.md#24649
ishaan-berri merged 1 commit intomainfrom
ishaan-berri-patch-1

Conversation

@ishaan-berri
Copy link
Copy Markdown
Contributor

Relevant issues

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/test_litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem
  • I have requested a Greptile review by commenting @greptileai and received a Confidence Score of at least 4/5 before requesting a maintainer review

Delays in PR merge?

If you're seeing a delay in your PR being merged, ping the LiteLLM Team on Slack (#pr-review).

CI (LiteLLM team)

CI status guideline:

  • 50-55 passing tests: main is stable with minor issues.
  • 45-49 passing tests: acceptable but needs attention
  • <= 40 passing tests: unstable; be careful with your merges and assess the risk.
  • Branch creation CI run
    Link:

  • CI run for the last commit
    Link:

  • Merge / cherry-pick CI run
    Links:

Type

🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test

Changes

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
litellm Ready Ready Preview, Comment Mar 26, 2026 10:24pm

Request Review

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@ishaan-berri ishaan-berri merged commit 6153793 into main Mar 26, 2026
32 of 36 checks passed
@ishaan-berri ishaan-berri deleted the ishaan-berri-patch-1 branch March 26, 2026 22:22
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 26, 2026

Greptile Summary

This PR adds a new entry to the OSS Adopters section of README.md by inserting an <img> tag for a new company logo. The change is purely cosmetic/documentation and has no impact on runtime code.\n\n- New adopter logo inserted between the Stripe and Google ADK entries in the OSS Adopters table.\n- The alt attribute for the new image is set to \"image\" (a generic placeholder) rather than the actual company/product name, which reduces accessibility and differs from the convention used by all other entries in the table.

Confidence Score: 4/5

Safe to merge after updating the alt attribute to the actual company name.

This is a one-line documentation change with no code impact. The only issue is the generic alt="image" placeholder, which is a minor accessibility/style concern that should be fixed for consistency with the rest of the table.

README.md — the alt attribute on the new image needs the correct company name.

Important Files Changed

Filename Overview
README.md Adds a new OSS adopter logo to the table; the image alt attribute is set to the placeholder value "image" instead of the actual company name.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[README.md OSS Adopters Table] --> B[Existing entries: Stripe, Google ADK, Greptile, OpenHands, Netflix, OpenAI Agents SDK]
    A --> C[New entry added: unknown company logo\nalt='image' - placeholder needs update]
    C --> D{alt attribute}
    D -->|Current: 'image'| E[❌ Generic - accessibility issue]
    D -->|Should be: Company Name| F[✅ Descriptive - consistent with other entries]
Loading

Reviews (1): Last reviewed commit: "Update README.md" | Re-trigger Greptile

Comment thread README.md
<table>
<tr>
<td><img height="60" alt="Stripe" src="https://github.com/user-attachments/assets/f7296d4f-9fbd-460d-9d05-e4df31697c4b" /></td>
<td><img height="60" alt="image" src="https://github.com/user-attachments/assets/436fca71-988b-40bb-b5fe-8450c80fdbd0" /></td>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 Non-descriptive alt attribute

The alt attribute is set to "image", which is a generic placeholder. All other entries in this table use the actual company or product name (e.g. "Stripe", "Google ADK", "Greptile", "OpenHands"). Please update this to the name of the adopter being added.

@codspeed-hq
Copy link
Copy Markdown
Contributor

codspeed-hq bot commented Mar 26, 2026

Merging this PR will not alter performance

✅ 16 untouched benchmarks


Comparing ishaan-berri-patch-1 (b69ce9f) with main (6153793)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (b69ce9f) during the generation of this report, so 6153793 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

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.

2 participants