Repository navigation
Update Service Levels docs to match the current UI - #2459
Conversation
There was a problem hiding this comment.
All reported issues were addressed across 2 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Team Run ID: 📒 Files selected for processing (1)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
Included review availability: Your plan provides up to 8 included reviews per hour; 5 remain after this review. 📝 WalkthroughWalkthroughThe documentation clarifies metric target semantics and lists the experimental Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to The supplied material identifies no specific documentation error or broken link. A hosted preview was not available for verification, but no actionable merge-blocking risk is established. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
Summary
Some statements in the Service Levels guide no longer match the Logfire UI. This PR corrects them:
logfire_sloresource. The Infrastructure as Code resource list and the Service Levels guide now mention it. The docs name the resource only and do not describe its attributes.The changes do not touch the burn-rate tier section, which #2458 updates separately. The two PRs merge cleanly in either order.
Preview
I built both pages locally with the unified docs preview (
make docs-serveflow), together with #2458. Both pages render, and every link on them resolves: the new links to Infrastructure as Code, SQL Workbench and Live view, the in-page anchors, and the Terraform, OpenTofu and Pulumi registry links. Please still check the hosted preview.