docs: correctly roll intro into single OpenML Server nav item#254
docs: correctly roll intro into single OpenML Server nav item#254Gokul-social wants to merge 2 commits intoopenml:mainfrom
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Repository UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughThe mkdocs.yml navigation was updated: the top-level nav label for 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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 |
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The nav reordering moves
Changesabove theContributingsection (previously it was below); double-check that this new position matches the intended sidebar hierarchy and UX, since the issue description only mentions deduplication, not reordering.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The nav reordering moves `Changes` above the `Contributing` section (previously it was below); double-check that this new position matches the intended sidebar hierarchy and UX, since the issue description only mentions deduplication, not reordering.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
There was a problem hiding this comment.
Pull request overview
This PR resolves issue #208 by restructuring the MkDocs navigation to consolidate the intro page under a single "OpenML Server" top-level navigation item, matching the MkDocs Material design pattern referenced in the issue. The change improves the documentation structure by eliminating duplicate sidebar entries and creating a clearer hierarchy.
Changes:
- Renamed the "Intro" navigation item to "OpenML Server" to serve as the main entry point
- Relocated the "Changes" item to appear before the "Contributing" section for better logical grouping
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Fixes #208
This PR updates the MkDocs navigation structure to match the intended layout discussed in the issue and PR #242.
Changes:
index.mdThis ensures only one "OpenML Server" item appears in the sidebar and matches the intended MkDocs Material layout.