Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

Add a print_message() or similar function in admin CLI #746

Closed
govint opened this issue Nov 17, 2016 · 1 comment
Closed

Add a print_message() or similar function in admin CLI #746

govint opened this issue Nov 17, 2016 · 1 comment

Comments

@govint
Copy link
Contributor

govint commented Nov 17, 2016

With each change that goes in for the admin CLI there is a chance that Python 3.5 incompatible changes get checked in - especially try - except blocks (uses except (exception name), e:) and print statements.

There are multiple places where new print messages keep getting added and those break the admin CLI on VSphere 6.5 (Python 3.5).

Instead define a new print_message() function that is called everywhere in the CLI code and it in turn uses the correct syntax for printing.

@govint
Copy link
Contributor Author

govint commented Nov 23, 2016

Closing, may be just rely on using a coding guideline to be Python 3.5 compliant when making changes.

@govint govint closed this as completed Nov 23, 2016
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

No branches or pull requests

1 participant