Skip to content
This repository was archived by the owner on Oct 6, 2025. It is now read-only.

Conversation

@ArthurFlag
Copy link
Contributor

@ArthurFlag ArthurFlag commented May 21, 2025

Migrate manually maintained cli docs to model-cli

Copy link
Contributor

@xenoscopic xenoscopic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM overall, just a had a few suggestions.

@ArthurFlag ArthurFlag requested a review from xenoscopic May 22, 2025 07:58
@crazy-max
Copy link
Member

https://github.com/docker/model-cli/actions/runs/15181080811/job/42690614593#step:3:409

0.549 ERROR: Docs result differs. Please update with "make docs"
0.563  M docs/reference/docker_model.yaml
0.563  M docs/reference/model.md

@ArthurFlag
Copy link
Contributor Author

I already ran make docs in my last commit 🤔

@crazy-max
Copy link
Member

I already ran make docs in my last commit 🤔

Ah I saw ci job failed but was probably due to https://www.githubstatus.com/incidents/cbdzqm5fw0fm

@xenoscopic xenoscopic merged commit 15d0078 into docker:main May 27, 2025
3 checks passed
ArthurFlag added a commit to docker/docs that referenced this pull request Jun 2, 2025
## Description

Adds model-cli reference docs.

## Related issues or tickets

* docker/model-cli#62
* docker/model-cli#69

## Reviews

<!-- Notes for reviewers here -->
<!-- List applicable reviews (optionally @tag reviewers) -->

- [ ] Technical review
- [ ] Editorial review
- [ ] Product review

---------

Signed-off-by: CrazyMax <[email protected]>
Co-authored-by: ArthurFlag <[email protected]>
sarahsanders-docker pushed a commit to sarahsanders-docker/docs that referenced this pull request Jun 4, 2025
## Description

Adds model-cli reference docs.

## Related issues or tickets

* docker/model-cli#62
* docker/model-cli#69

## Reviews

<!-- Notes for reviewers here -->
<!-- List applicable reviews (optionally @tag reviewers) -->

- [ ] Technical review
- [ ] Editorial review
- [ ] Product review

---------

Signed-off-by: CrazyMax <[email protected]>
Co-authored-by: ArthurFlag <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants