Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request performs a major documentation restructure, removing all existing documentation content and replacing it with minimal API reference documentation for Heimdall and Odin components.
Changes:
- Removes all existing documentation (getting started, features, best practices, benchmarking guides)
- Removes all static assets (logos, images)
- Adds new API reference documentation for Heimdall and Odin
- Updates AGENTS.md with simplified documentation guidelines
- Converts category metadata from JSON to YAML format
- Modifies package-lock.json dependency structure
Reviewed changes
Copilot reviewed 31 out of 48 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| website/docs/reference/heimdall/api-reference.mdx | New API reference for Heimdall scheduler with Kubernetes API documentation |
| website/docs/reference/odin/api-reference.mdx | New API reference for Odin inference service with Kubernetes API documentation |
| website/docs/reference/heimdall/category.yaml | Category metadata for Heimdall reference docs |
| website/docs/reference/odin/category.yaml | Category metadata for Odin reference docs |
| website/docs/reference/category.yaml | Category metadata for Reference section |
| website/AGENTS.md | Simplified documentation guidelines and formatting rules |
| website/.gitignore | Updated with comprehensive Node.js gitignore template |
| website/package-lock.json | Removed "peer": true flags from multiple dependencies |
| website/static/moreh-logo-white.svg | Modified SVG with new color class and minor formatting changes |
| Multiple .mdx and image files | Deleted all existing documentation content and assets |
Files not reviewed (1)
- website/package-lock.json: Language not supported
Comments suppressed due to low confidence (1)
website/package-lock.json:18267
- Multiple dependencies have had their
"peer": trueflag removed in package-lock.json. This change converts peer dependencies into direct dependencies, which could significantly increase the bundle size and potentially cause version conflicts. Verify that this change is intentional and that all affected packages (@algolia/client-search, @babel/core, @docusaurus/core, react, react-dom, webpack, etc.) should indeed be direct dependencies rather than peer dependencies. This appears to be a substantial change to the dependency structure that may have been unintentionally introduced.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.