Add comprehensive help text for pyAnalyze and pyAnalyzeLaurel commands#405
Add comprehensive help text for pyAnalyze and pyAnalyzeLaurel commands#405tautschnig wants to merge 9 commits intomainfrom
Conversation
- Rename argument from 'file' to 'ion_file' for clarity - Add detailed description explaining the command's purpose - Document both arguments with clear usage information - Specify verbose mode options (1 for verbose, 0 for concise)
There was a problem hiding this comment.
Pull request overview
Improves the CLI help/usage text for the pyAnalyze subcommand to better explain expected inputs and verbosity behavior, and adjusts --help output formatting to accommodate longer help strings.
Changes:
- Renames
pyAnalyzepositional argument label fromfiletoion_file. - Expands
pyAnalyzehelp text with detailed argument descriptions and verbose-mode behavior. - Changes
strata --helpoutput to print command usage and help on separate lines.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Expanding the proposed changes to a discussion: the new output, for me, looks like this: Questions:
|
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Aaron Tomb <aarotomb@amazon.com>
Agreed. Not that I know of, but I think you can check the issues as well as I can.
That's probably a good idea to rethink the UI. This started as some utilities for the DDM and has grown.
Is that a request to rename I'll note I find it a bit distracting to have discussions of unrelated issues in PRs. Discoverability of this seems low. |
Fair, I'll work on this.
Created #447 to track this.
No, it's me asking for help in understanding what those subcommands are actually doing. I'll create another issue. |
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.