Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
|
Greptile SummaryThis is a documentation-only PR that reorganizes the benchmarks page (moving machine specs and result tables to the top for better first-impression impact) and updates navigation in Confidence Score: 5/5Safe to merge — documentation-only changes with no code impact. All findings are P2 style suggestions (link label wording, sidebar grouping). No code, logic, or security changes are involved. docs/my-website/sidebars.js — the load-testing docs are now split across two sidebar locations.
|
| Filename | Overview |
|---|---|
| README.md | Updates the "Website" header link to point to /ai-gateway instead of the site root. |
| docs/my-website/docs/benchmarks.md | Reorganizes content order — moves machine specs, configuration, and benchmark result tables to the top, and adds a benchmarking script link. |
| docs/my-website/sidebars.js | Removes the "Load Testing" category; benchmarks becomes a standalone top-level item while load_test_advanced/sdk/rpm are moved to the "Extras" section, splitting previously grouped docs. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Sidebar Navigation] --> B[Routing]
A --> C["benchmarks (standalone)"]
A --> D[Contributing]
A --> E[Extras]
E --> F[data_retention]
E --> G[security_encryption_faq]
E --> H[migration_policy]
E --> I[load_test_advanced]
E --> J[load_test_sdk]
E --> K[load_test_rpm]
E --> L[Projects built on LiteLLM]
style C fill:#f9f,stroke:#333
style I fill:#ffa,stroke:#333
style J fill:#ffa,stroke:#333
style K fill:#ffa,stroke:#333
Reviews (1): Last reviewed commit: "docs: readme tweak" | Re-trigger Greptile
| </p> | ||
| </p> | ||
| <h4 align="center"><a href="https://docs.litellm.ai/docs/simple_proxy" target="_blank">LiteLLM Proxy Server (AI Gateway)</a> | <a href="https://docs.litellm.ai/docs/enterprise#hosted-litellm-proxy" target="_blank"> Hosted Proxy</a> | <a href="https://litellm.ai/enterprise"target="_blank">Enterprise Tier</a> | <a href="https://litellm.ai/" target="_blank">Website</a></h4> | ||
| <h4 align="center"><a href="https://docs.litellm.ai/docs/simple_proxy" target="_blank">LiteLLM Proxy Server (AI Gateway)</a> | <a href="https://docs.litellm.ai/docs/enterprise#hosted-litellm-proxy" target="_blank"> Hosted Proxy</a> | <a href="https://litellm.ai/enterprise"target="_blank">Enterprise Tier</a> | <a href="https://www.litellm.ai/ai-gateway" target="_blank">Website</a></h4> |
There was a problem hiding this comment.
"Website" link points to a product page, not the site root
The link label is Website but the destination is now https://www.litellm.ai/ai-gateway — a specific product page. Visitors clicking "Website" typically expect the homepage. Consider either updating the label to AI Gateway or keeping the URL as the site root (https://www.litellm.ai/).
| @@ -1097,6 +1088,9 @@ const sidebars = { | |||
| "data_retention", | |||
| "proxy/security_encryption_faq", | |||
| "migration_policy", | |||
| "load_test_advanced", | |||
| "load_test_sdk", | |||
| "load_test_rpm", | |||
There was a problem hiding this comment.
Load-testing docs split across two unrelated locations
benchmarks is now a standalone top-level item while load_test_advanced, load_test_sdk, and load_test_rpm land in the "Extras" section alongside data-retention and security pages. Users looking for load-testing material will have to search in two different parts of the sidebar. Keeping them together (either as a flat group at the top level or as siblings in "Extras") would avoid the split.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Relevant issues
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
tests/test_litellm/directory, Adding at least 1 test is a hard requirement - see detailsmake test-unit@greptileaiand received a Confidence Score of at least 4/5 before requesting a maintainer reviewDelays 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)
Branch creation CI run
Link:
CI run for the last commit
Link:
Merge / cherry-pick CI run
Links:
Screenshots / Proof of Fix
Type
🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test
Changes