-
Notifications
You must be signed in to change notification settings - Fork 9
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
Make CLI commands available as cubids <command>
#268
Conversation
Here's what the CLI documentation looks like: https://cubids--268.org.readthedocs.build/en/268/usage.html#command-line-tools |
cubids <command>
I'm not sure that the PyPi deployment job will succeed, but I can address that in a followup PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fantastic, thank you!!
cubids-datalad-save = "cubids.cli:cubids_datalad_save" | ||
cubids-print-metadata-fields = "cubids.cli:cubids_print_metadata_fields" | ||
cubids-remove-metadata-fields = "cubids.cli:cubids_remove_metadata_fields" | ||
cubids = "cubids.cli:_main" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this new configuration
|
||
This will print the instructions for using the command line interface in your command line. | ||
|
||
.. argparse:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also a great idea
Thanks @mattcieslak! |
This builds off of #266, so that should be addressed first.
Changes proposed:
cubids
command (which we can call--version
and--help
on), with each of the regular commands as subcommands.cubids apply
andcubids-apply
, butcubids-apply
will raise a deprecation warning.