Skip to content

Conversation

@siddharth7
Copy link
Contributor


This checklist is used to make sure that common guidelines for a pull request are followed.

  • The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

  • I adhere to the Command Guidelines.

@siddharth7 siddharth7 force-pushed the user/t-sabhal/sapHana branch from c7f3db0 to e6bd3f5 Compare September 20, 2019 07:09
@qwordy qwordy self-requested a review September 20, 2019 07:28
def transform_workload_policy_show(result):
columns = []
columns.append(('Name', result['name']))
columns.append(('WorkloadType', result['properties']['workLoadType']))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

columns = [
    ('Name', result['name']),
    ('WorkloadType', result['properties']['workLoadType'])
]

Maybe more readable

@yonzhan
Copy link
Collaborator

yonzhan commented Oct 14, 2019

@qwordy please help deal with this PR.

@qwordy qwordy closed this Oct 15, 2019
@qwordy qwordy reopened this Oct 15, 2019
@qwordy
Copy link
Member

qwordy commented Oct 15, 2019

Some checks were not successful. Remember to fix.

@mmyyrroonn
Copy link
Contributor

@siddharth7 Hello. Since it's a huge change, did you schedule a quick meeting before with any one of CLI team? We can have a quick meeting if you want.

@mmyyrroonn
Copy link
Contributor

Close it since a new pr appears. #11066

@mmyyrroonn mmyyrroonn closed this Dec 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants