Skip to content

docs: remove phone numbers from readme and docs#24587

Merged
ishaan-berri merged 1 commit intomainfrom
remove-phone-numbers
Mar 25, 2026
Merged

docs: remove phone numbers from readme and docs#24587
ishaan-berri merged 1 commit intomainfrom
remove-phone-numbers

Conversation

@ishaan-berri
Copy link
Copy Markdown
Contributor

Relevant issues

N/A

Pre-Submission checklist

  • My PR's scope is as isolated as possible, it only solves 1 specific problem

No tests needed — docs-only change.

Type

📖 Documentation

Changes

Removes personal phone numbers from README.md, cookbooks, and ~16 doc pages (observability integrations, proxy docs, troubleshoot). The numbers appeared in a contact section across the docs.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 25, 2026

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

Project Deployment Actions Updated (UTC)
litellm Ready Ready Preview, Comment Mar 25, 2026 7:42pm

Request Review

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 25, 2026

Greptile Summary

This PR removes personal phone numbers (+1 (770) 8783-106 / +1 (412) 618-6238) from the "contact us" sections that appeared across 19 documentation files, including README.md, cookbook READMEs, observability integration docs, proxy docs, and the troubleshooting guide. The remaining contact options (calendar link, Discord, Slack, and email addresses) are preserved.

  • Purely documentation change — no code, tests, or configuration files are modified.
  • The removal is consistent and complete across all identified locations.
  • In docs/my-website/docs/proxy/pass_through.md and docs/my-website/docs/troubleshoot.md, removing the phone number line leaves a superfluous blank line in the contact block, but this is cosmetically harmless in rendered Markdown.

Confidence Score: 5/5

  • Safe to merge — documentation-only change with no impact on code, tests, or configuration.
  • All 19 changed files are Markdown documentation. The edit is a straightforward line deletion repeated uniformly across the repo. There are no logic, security, or behavioral concerns.
  • No files require special attention.

Important Files Changed

Filename Overview
README.md Removes phone number line from the contact section; no issues.
cookbook/codellama-server/README.MD Removes phone number line from contact section; clean removal.
cookbook/litellm_proxy_server/readme.md Removes phone number line from contact section; clean removal.
docs/my-website/docs/proxy/pass_through.md Removes phone number line; leaves a minor extra blank line between Discord and email entries, but harmless in rendered markdown.
docs/my-website/docs/troubleshoot.md Removes phone number line; leaves a minor extra blank line between Discord/Slack entries and the email entry, harmless in rendered markdown.
docs/my-website/docs/proxy_server.md Removes phone number line from contact section; no issues.
docs/my-website/docs/observability/langfuse_integration.md Removes phone number line from contact section; no issues.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[19 Documentation Files] --> B[Remove phone number line\n📞 +1 770 8783-106 / +1 412 618-6238]
    B --> C[Contact section after change]
    C --> D[Schedule Demo 👋 calendly link]
    C --> E[Community Discord 💭]
    C --> F[Community Slack 💭]
    C --> G[Emails ✉️ [email protected] / [email protected]]
Loading

Reviews (1): Last reviewed commit: "docs: remove phone numbers from readme a..." | Re-trigger Greptile

@codspeed-hq
Copy link
Copy Markdown
Contributor

codspeed-hq bot commented Mar 25, 2026

Merging this PR will not alter performance

✅ 16 untouched benchmarks


Comparing remove-phone-numbers (a2f02aa) with main (67609e0)

Open in CodSpeed

@ishaan-berri ishaan-berri merged commit 90b850e into main Mar 25, 2026
37 of 38 checks passed
@ishaan-berri ishaan-berri deleted the remove-phone-numbers branch March 26, 2026 22:30
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