openstack/python-openstackclient
Code Issues Proposed changes
375 Commits 7 Branches 64 Tags
cf8506d2bd
Commit Graph

9 Commits

Author SHA1 Message Date
Andreas Jaeger
5043293156 Fix some help strings 
This fixes some errors and inconsistencies I found reviewing the
help strings:
* Capitalize help strings
* Add missing space between words (in multi-line strings)
* Improve wording

Change-Id: I2fb31ab4191c330146e31c1a9651115a6657769a
 2014-02-21 19:40:41 +01:00
Dean Troyer
dfb0e3e3c1 Begin Python 3 compatability 
* use six.iteritems()
* replace basestring with six.string_types
* convert print statements to functions (they're all debugging and should
  be removed eventually anyway)

* clean up OpenStack copyright: LLC -> Foundation

Change-Id: Icb14212bcb408e63816bfec3922a697bc1a6c946
 2013-07-29 19:12:29 -05:00
Steve Martinelli
1a0d5ccc68 Remove api = apiName calls from each method 
As discussed in https://review.openstack.org/#/c/36352/ for each
command, we were setting api = identity or volume... etc,
this was for an old way of calling commands that are is no longer
used.

Also removed openstackclient/common/command.py

Change-Id: I2705f35d343f2ae729dc22d6aed0b852b2f8ca19
 2013-07-12 11:07:33 -05:00
Josh Kearney
b26cb5bf68 Upgraded to PEP8 1.3.3 to stay aligned with Nova, etc. 
Made all the necessary changes to pass new PEP8 standards.

Also cleaned up docstrings to conform to the HACKING stanards.

Change-Id: Ib8df3030da7a7885655689ab5da0717748c9edbe
 2013-01-31 13:31:41 -06:00
Josh Kearney
0a4912f9fb Standardize on a copyright header and ensure all files have them. 
Change-Id: I64812bca01ca655c9cf9239a0daea84907082a29
 2013-01-24 12:02:23 -06:00
Josh Kearney
df34db8afb Remove incorrect 'raise'. 
Fixes bug 1096102.

Change-Id: Ibcdccd949566f47bb516c7562149d6b0100fce18
 2013-01-22 20:22:40 -06:00
Dean Troyer
697a5ac6cb Refactor cliff.Command.run() 
* All commands now perform their action in take_action().  Those producing
  output are derived from DisplayCommandBase.

Change-Id: Ic93ba9a2ad449d84242b6aa8624b41379c4fb79a
 2012-06-20 10:03:01 -05:00
Clark Boylan
6a8302b532 Fix pep8 issues. 
Fix several pep8 issues and pin the pep8 test to pep8 version 1.1.
This should prevent future changes from being unmergable by the pep8
gate after a pep8 upgrade.

Change-Id: I4678a9179579fb5c7afe795fb43a8a89a99ad717
 2012-06-13 10:49:43 -07:00
Dean Troyer
0c4e131c6e Add endpoint CRUD commands 
* add {create|delete|list|show} endpoint commands

Change-Id: Ife9fa789d5818d63288b09687b43d802b1b97858
 2012-05-18 10:35:20 -05:00