fix: point _index.md TiDB Self-Managed links to stable instead of unpublished v8.5 - #23847
Conversation
…hed v8.5 The ja/tidb/v8.5/tidb-configuration-file route is not yet published and returns 404; ja/tidb/stable/tidb-configuration-file resolves (200). Point the link at stable until the v8.5 Japanese docs are published. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 4 included reviews per hour; 0 remain after this review. 📝 WalkthroughWalkthroughJapanese TiDB Self-Managed navigation links now use ChangesJapanese navigation links
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The navigation links are updated to stable documentation paths, with no remaining merge-blocking issue identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
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 |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
… v8.5 The other 37 ja/tidb/v8.5/... links in this file 404 for the same reason as the tidb-configuration-file link fixed in the previous commit: the Japanese v8.5 docs are not published yet. Point all of them at stable, which resolves (200) for every path. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
@coderabbitai full review |
|
|
@coderabbitai full review |
✅ Action performedFull review finished. |
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Advanced
Run ID: 19d254e6-0f9e-4430-9534-b05d51e2fcb6
📒 Files selected for processing (1)
_index.md
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
[LGTM Timeline notifier]Timeline:
|
What is changed, added or deleted? (Required)
_index.md's "TiDB Self-Managed" page had 38 reference links pointing tohttps://docs.pingcap.com/ja/tidb/v8.5/..., but the Japanese v8.5 docs arenot published yet, so every one of these links 404s. This PR points all
38 links at
stableinstead, which resolves correctly for every path.Verified before/after for all 36 unique paths used in this file:
https://docs.pingcap.com/ja/tidb/v8.5/<path>->404(all 36)https://docs.pingcap.com/ja/tidb/stable/<path>->200(all 36)Originally flagged by CodeRabbit on PR #23832 (a separate configuration-terminology PR) for just one of these 38 links (
tidb-configuration-file); a repo-wide check found only 2 other files reference/ja/tidb/v8.5/at all, and both arealiases:front-matter entries (a page's own future-alias declaration, not a clickable link), so out of scope here.Which TiDB version(s) do your changes apply to? (Required)
What is the related PR or file link(s)?
AI agent involvement
Do your changes match any of the following descriptions?
🤖 Generated with Claude Code
Summary by CodeRabbit