Uploaded image for project: 'Commons CLI'
  1. Commons CLI
  2. CLI-239

Patch: Required Options Not Using Generics

Agile BoardAttach filesAttach ScreenshotAdd voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Trivial
    • Resolution: Unresolved
    • None
    • 2.0
    • None
    • None

    Description

      The way in which required options were being handled involved a list blended with Options and Strings. I have simplified the process and now the list only contains Options.

      With this patch, OptionGroup now enforces that all Option objects added to it match its own "required" level.

      The unit tests are broken with these changes though.

      Attachments

        1. DefaultParser.java.patch
          1 kB
          David Mollitor
        2. OptionGroup.java.patch
          0.8 kB
          David Mollitor
        3. Options.java.patch
          2 kB
          David Mollitor

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            belugabehr David Mollitor
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:

              Slack

                Issue deployment