[doc] chore: remove duplicate moe-comm-overlap skill#2943
[doc] chore: remove duplicate moe-comm-overlap skill#2943
Conversation
The moe-comm-overlap skill was fully subsumed by the more thorough expert-parallel-overlap skill. Remove the duplicate and clean up stale references in docs/skills-index.md and docs/training/communication-overlap.md. Signed-off-by: yaoyu-33 <yaoyu.094@gmail.com> Made-with: Cursor
|
/ok to test 0c26d75 |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
💤 Files with no reviewable changes (4)
📝 WalkthroughWalkthroughRemoved MoE expert-parallel communication overlap skill documentation and metadata from the documentation structure, including deletion of the SKILL.md and card.yaml files and removal of all references from navigation indices and related documentation. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 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 Tip CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required. |
Summary
moe-comm-overlapskill (SKILL.md + card.yaml) which was fully subsumed by the more thoroughexpert-parallel-overlapskilldocs/skills-index.md(toctree entry) anddocs/training/communication-overlap.md(related-docs links)Test plan
docs/skills-index.mdtoctree renders without broken referencesdocs/training/communication-overlap.mdrelated-docs links are all validMade with Cursor
Summary by CodeRabbit