Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Cylc Message Help Contains two patterns of use #5921

Draft
wants to merge 1 commit into
base: 8.3.x
Choose a base branch
from
Draft

Cylc Message Help Contains two patterns of use #5921

wants to merge 1 commit into from
+115 −23

Commits on Oct 4, 2024

  1. Allow CylcOptionParser to display docs for a command which 

    might have multiple different valid arguments.

e.g. Cylc message can be either
cylc message -- [WORKFLOW JOB [[SEVERITY:]MESSAGE ...]]]
__or__
cylc message --	[[SEVERITY:]MESSAGE ...]]]

- Refactored part of the __init__ method of CylcOptionParser.
  into a method for ease of testing.
- Added unit tests to get complete coverage of new method.

Response to review: the ... from the second usage.
Added extra information to the descriptions of the args.

remove cli-help test
    @wxtim
    wxtim committed Oct 4, 2024
    Configuration menu
    Copy the full SHA
    1fd2de3 View commit details
    Browse the repository at this point in the history