Fix test suite documentation links#1997
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (11)
📝 WalkthroughWalkthroughSuite index generation and corresponding current/versioned documentation tables now link directly to same-directory Markdown test pages using ChangesTest documentation links
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Up to standards ✅🟢 Issues
|
What changed
./<test-id>.md).Why
Suite index links were emitted as extensionless parent-relative URLs such as
../EIDSCA.AF01. When a suite page was reached through client-side navigation without a trailing slash, the browser resolved those links one directory too high (for example,/docs/EIDSCA.AF01instead of/docs/tests/EIDSCA.AF01), resulting in a 404.Including the Markdown extension lets Docusaurus resolve each source document to the correct absolute, version-aware permalink during compilation, independent of trailing-slash behavior.
Impact
Individual test links from all generated suite overview pages now open the correct test documentation. This fixes the reported EIDSCA and ORCA failures and prevents the same issue in the other suites.
Validation
/docs/tests/eidscato/docs/tests/EIDSCA.AF01./docs/tests/orcato/docs/tests/ORCA.100.node --check.useDocsSidebarcontext error on/docs/commands/and/docs/next/commands/.Summary by CodeRabbit