fix(upload): help example fails with missing uniqueIdentifier#64
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. To trigger a review, include ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Free Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Note 🎁 Summarized by CodeRabbit FreeYour organization has reached its limit of developer seats under the Pro Plan. For new users, CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please add seats to your subscription by visiting https://app.coderabbit.ai/login.If you believe this is a mistake and have available seats, please assign one to the pull request author through the subscription management page using the link above. Comment |
There was a problem hiding this comment.
✅ Ready to approve
The changes are small and scoped to CLI UX/docs and required-flag enforcement, with only minor wording nits noted.
Note: this review does not count toward required approvals for merging.
Pull request overview
Updates the runware model upload CLI command help and generated docs so the documented usage matches current required inputs (notably --unique-identifier, and now --air), preventing copy/paste example failures.
Changes:
- Updated
model uploadhelp examples to include--unique-identifierand clarified--airplaceholder usage. - Updated flag help text for
--air/--unique-identifierand markedair+unique-identifieras required flags. - Regenerated the corresponding CLI markdown doc to match the new help output.
File summaries
| File | Description |
|---|---|
| internal/cmd/model/upload.go | Updates CLI examples/help text and enforces required flags for upload requests. |
| docs/runware_model_upload.md | Updates generated CLI reference to match the new examples/flag descriptions. |
Copilot's findings
- Files reviewed: 2/2 changed files
- Comments generated: 2
Note
Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Resolving: RUN-10703