Repository navigation
docs: fix stale examples and checkpoint documentation audit - #5974
Conversation
Co-Authored-By: GPT-6 <noreply@openai.com>
Co-Authored-By: GPT-6 <noreply@openai.com>
Co-Authored-By: GPT-6 <noreply@openai.com>
Co-Authored-By: GPT-6 <noreply@openai.com>
Co-Authored-By: GPT-6 <noreply@openai.com>
Co-Authored-By: GPT-6 <noreply@openai.com>
Co-Authored-By: GPT-6 <noreply@openai.com>
Co-Authored-By: GPT-6 <noreply@openai.com>
|
Checkpoint: concrete fixes are pushed at (written by GPT-6.1-Sol) |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 27 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Problem
The production setup still used the retired relay
auth.keysetting. Codec support, Rust driver setup, snapshot staleness, and several API references also lagged current main.Approach
Fix those pages and adjacent snapshot API comments. Replace unavailable API destinations with verified source links. Keep the audit quest open with its coverage baseline and remaining work; this run is incomplete, so no recurring successor is added.
Iterate round: merged
origin/main, resolving thedoc/bin/cli.mdconflict onto main's rewritten MPEG-TS paragraph. Dropped the "(up to 7.1)" qualifier on TS Opus, since the importer's explicit channel layout (channel_config_code0x81) also accepts more than eight channels.Impact
Decisions
Test(test_testing_clock.py::test_a_pending_timer_waits_for_advance,MoqError.Protocol: old): ✅ treat as a pre-existing flake outside this docs PR / fix it here (unrelated scope). It also failed in the merge queue for moq-dev/moq#5919 and onfix/keylog-opt-in.Alternatives
Keep valid current source references until hosted API documentation is available, rather than retaining dead links or linking to an older API.
Validation
just checkon the changed packages (doc site, JS packages) andquest checkpassed after the merge.moqbuild passed; 34 help contexts and 37 complete supported shell examples checked.Follow-ups
quest/m0/audit-docs.md: exported-symbol and Cross-Package Sync walk, remaining refused links, optional-feature builds. The successor is added once that run completes.(Written by Claude Opus 5.5)
🤖 Generated with Claude Code