Skip to content

docs: fix dead LlamaIndex observability link in README - #4670

Open
sarankumar1325 wants to merge 1 commit into
traceloop:mainfrom
sarankumar1325:docs/fix-llamaindex-observability-link
Open

sarankumar1325 wants to merge 1 commit into
traceloop:mainfrom
sarankumar1325:docs/fix-llamaindex-observability-link

Conversation

@sarankumar1325

@sarankumar1325 sarankumar1325 commented Oct 29, 2026 •

Copy link
Copy Markdown

The LlamaIndex entry in the README's "Frameworks" list points to https://docs.llamaindex.ai/en/stable/module_guides/observability/observability.html#openllmetry, which now returns 404. LlamaIndex moved its docs to developers.llamaindex.ai.

This PR points the link at the current observability guide, https://developers.llamaindex.ai/python/framework/module_guides/observability/#openllmetry. That page returns 200 and still has the openllmetry section anchor.

  • I have added tests that cover my changes. (README link only, so no tests apply)
  • If adding a new instrumentation or changing an existing one, I've added screenshots from some observability platform showing the change. (n/a)
  • PR name follows conventional commits format.
  • (If applicable) I have updated the documentation accordingly.

Summary by CodeRabbit

  • Documentation
    • Updated the LlamaIndex observability link in the README to point to the Python framework documentation.

@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.

@coderabbitai

coderabbitai Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 698ba973-1fb9-4af3-aea8-ff1020a72fe9
Commits

Reviewing files that changed from the base of the PR and between 8d9c93f and 3337f6f.

Files selected for processing: 1
  • README.md

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review.

Walkthrough

The README’s LlamaIndex observability link now points to the Python framework observability guide.

Changes: LlamaIndex documentation link

Layer / File(s) Summary
Update observability link
README.md
The LlamaIndex observability link now points to the Python framework observability guide.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: nirga

Merge Risk: Minimal · up to 3337f

The documentation link change is ready to merge.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the README change: fixing the dead LlamaIndex observability link.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

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