Skip to content

Conversation

@DannyBen
Copy link
Member

@DannyBen DannyBen commented Nov 3, 2025

cc #679


This PR adds the ability to disable the --help flag when using catch_all.
It is done by adding another option to the catch_all hash syntax:

commands:
- name: docker
  help: Pass all arguments to the docker cli
  catch_all:
    label: params
    help: Parameters to pass to docker
    required: false
    catch_help: true   # <= this

Tasks

  • Implementation
  • Specs
  • Examples (internal fixtures)
  • Decide between this and command.helpless implementation (👇)
  • JSON Schema
  • Docs 👉 Source | Preview

About choosing command.catch_all.catch_help over command.helpless

I have tried to implement the command.helpless: true approach, but on top of my original concerns - that this is and edg feature that gets "center stage" for no reason - it is also somewhat more involved to accomplish, due to the fact that there are quite a few edge cases that need to be adjusted to support it. So I am sticking to my original approach that states that there should be no reason to disable --help, unless it is in the context of catch_all.

@DannyBen DannyBen merged commit d9c0812 into master Nov 3, 2025
7 checks passed
@DannyBen DannyBen deleted the add/support-for-disabling-help-flag branch November 3, 2025 10:56
@DannyBen DannyBen added this to the 1.3.4 milestone Nov 3, 2025
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.

2 participants