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

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_r115726390
  
    --- Diff: aria/cli/commands/services.py ---
    @@ -38,6 +41,53 @@ def services():
         pass
     
     
    [email protected](name='show',
    +                  short_help='Display service information')
    [email protected]('service-name')
    [email protected]()
    [email protected]_all
    [email protected]_graph
    [email protected]_json
    [email protected]_yaml
    [email protected]_model_storage
    [email protected]_logger
    +def show(service_name, model_storage, all, graph, json, yaml, logger):
    +    """Show information for a specific service template
    +
    +    `SERVICE_NAME` is the name of the service to display information on.
    +    """
    +    logger.info('Showing service {0}...'.format(service_name))
    +    service = model_storage.service.get_by_name(service_name)
    +
    +    if json or yaml:
    +        all = True
    --- End diff --
    
    i'd rename the flag, it's both unclear and a python keyword. `show_all` or 
`all_info` might be better.


> 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