-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add support for subcommands #17
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This error was worded as an internal error, but can actually be displayed to the user, for example when a stray '=' appears. For example, if you typo "-ss = val" instead of "-s = val". Old: Unrecognized token type: {<equals-token>} New: Unexpected token: "="
These can both be computed dynamically and there is no worry about speed in this context.
This slot does not contain options. Certainly not in the <option> sense, and I think not in the usaul command-line sense either. Unfortunately this is an API change. Since the documentation is maintained elsewhere, for now I've defined a constant with the old name.
housel
reviewed
Feb 28, 2021
cgay
added a commit
to cgay/command-line-parser
that referenced
this pull request
Mar 4, 2022
cgay
added a commit
to cgay/command-line-parser
that referenced
this pull request
Mar 4, 2022
fraya
added a commit
to fraya/command-line-parser
that referenced
this pull request
Apr 9, 2024
Since issue dylan-lang#17 is fixed, update version of 'install-opendylan' action and use 'dylan-tool' to build the project.
fraya
added a commit
to fraya/command-line-parser
that referenced
this pull request
Apr 9, 2024
Since issue dylan-lang#17 is fixed, update version of 'install-opendylan' action and use 'dylan-tool' to build the project.
fraya
added a commit
to fraya/command-line-parser
that referenced
this pull request
Apr 9, 2024
- Add missing checkout action. - Since issue dylan-lang#17 is fixed, update version of 'install-opendylan' action and use 'dylan-tool' to build the project.
fraya
added a commit
to fraya/command-line-parser
that referenced
this pull request
Apr 9, 2024
- Add missing checkout action. - Since issue dylan-lang#17 is fixed, update version of 'install-opendylan' action and use 'dylan-tool' to build the project.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Docs are here.
<positional-option>
as a way to handle positional arguments just like pass-by-name arguments. The command line can now be fully described and the "usage" line is fully auto-generated.See doc for details.
I must admit that at some point there were so many changes I stopped trying to keep it minimal...
The main remaining TODO is to add support for subcommands to the
define command-line
macro.