Skip to content

fix: accept flags after positional args#137

Merged
tasuku43 merged 1 commit intomainfrom
fix-no-prompt
Feb 5, 2026
Merged

fix: accept flags after positional args#137
tasuku43 merged 1 commit intomainfrom
fix-no-prompt

Conversation

@tasuku43
Copy link
Owner

@tasuku43 tasuku43 commented Feb 5, 2026

Summary

  • Accept flags even when they appear after positional args.
  • Fixes unexpected usage: errors when running commands like:
    • gion manifest add --repo <repo> <WORKSPACE_ID> --no-prompt
    • gion manifest rm <WORKSPACE_ID> --no-prompt
    • gion apply --no-prompt

Details

  • Adds shared argument normalization to move flags before positionals (respecting --).
  • Applies it to manifest add, manifest rm, and manifest preset add/rm.
  • Adds global arg normalization so global flags work after the command (e.g. apply --no-prompt).

Tests

  • gofmt -w .
  • go test ./...
  • go vet ./...
  • go build ./...

@tasuku43 tasuku43 merged commit c48d497 into main Feb 5, 2026
2 checks passed
@tasuku43 tasuku43 deleted the fix-no-prompt branch February 5, 2026 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant