-
Notifications
You must be signed in to change notification settings - Fork 9
feat(ENG-9724): update generic docs #313
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds comprehensive documentation for Cloudsmith's Generic format support, enabling users to store and manage arbitrary files using path-based identification.
Changes:
- Added new Generic format documentation covering upload, download, HTML/JSON indexing, upstream proxying, and GPG signing
- Added Generic menu entry in the documentation navigation (correctly positioned alphabetically between Docker and Go)
- Cleaned up trailing whitespace in menu.json for improved code quality
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/content/menu.json | Added Generic format menu entry and removed trailing whitespace throughout the file |
| src/content/formats/generic-repository.mdx | New comprehensive documentation for Generic format including CLI/web upload, public/private download instructions, HTML/JSON indexing, upstream proxying configurations, and GPG signing support |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Just need to update the upstreams/index and should be good to go |
|
Example of what we added for the HuggingFace Format & Conda upstreams when they were added as new that we can use to check we include everything necessary: |
|
Updated the above with a screenshot from the new web-ui @apoclyps |
|
Hey there; could this text be moved up to just after the heading, used as an introduction?
|
Add docs outlining our support for Generic format