[ 
https://issues.apache.org/jira/browse/ARIA-148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16006110#comment-16006110
 ] 

ASF GitHub Bot commented on ARIA-148:
-------------------------------------

Github user ran-z commented on a diff in the pull request:

    https://github.com/apache/incubator-ariatosca/pull/107#discussion_r115934893
  
    --- Diff: aria/cli/core/aria.py ---
    @@ -40,16 +41,8 @@
     
     
     class MutuallyExclusiveOption(click.Option):
    -    """Makes options mutually exclusive. The option must pass a `cls` 
argument
    -    with this class name and a `mutually_exclusive` argument with a list of
    -    argument names it is mutually exclusive with.
    -
    -    NOTE: All mutually exclusive options must use this. It's not enough to
    -    use it in just one of the options.
    -    """
    -
         def __init__(self, *args, **kwargs):
    -        self.mutually_exclusive = set(kwargs.pop('mutually_exclusive', []))
    +        self.mutually_exclusive = kwargs.pop('mutually_exclusive', tuple())
    --- End diff --
    
    why change this from a set to another type?


> Enhance CLI "show" commands
> ---------------------------
>
>                 Key: ARIA-148
>                 URL: https://issues.apache.org/jira/browse/ARIA-148
>             Project: AriaTosca
>          Issue Type: Story
>            Reporter: Tal Liron
>            Assignee: Tal Liron
>
> For {{aria service-templates show}} and {{aria services show}} add support 
> for dumping, including graphs and type hierarchies, in JSON and YAML.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to