Merge "Fix PEP8 H405 errors"

2 files changed, 18 insertions, 6 deletions

diff --git a/openstackclient/common/commandmanager.py b/openstackclient/common/commandmanager.py
index aa238a23..08710c77 100644
--- a/openstackclient/common/commandmanager.py
+++ b/openstackclient/common/commandmanager.py
@@ -13,7 +13,7 @@
 #   under the License.
 #
 
-"""Modify Cliff's CommandManager"""
+"""Modify cliff.CommandManager"""
 
 import logging
 import pkg_resources
@@ -25,9 +25,12 @@ LOG = logging.getLogger(__name__)
 
 
 class CommandManager(cliff.commandmanager.CommandManager):
-    """Alters Cliff's default CommandManager behaviour to load additional
-       command groups after initialization.
+    """Add additional functionality to cliff.CommandManager
+
+    Load additional command groups after initialization
+    Add *_command_group() methods
     """
+
     def __init__(self, namespace, convert_underscores=True):
         self.group_list = []
         super(CommandManager, self).__init__(namespace, convert_underscores)
diff --git a/openstackclient/common/parseractions.py b/openstackclient/common/parseractions.py
index 644472d8..8f6008e2 100644
--- a/openstackclient/common/parseractions.py
+++ b/openstackclient/common/parseractions.py
@@ -19,9 +19,11 @@ import argparse
 
 
 class KeyValueAction(argparse.Action):
-    """A custom action to parse arguments as key=value pairs.
-    Ensures that dest is a dict
+    """A custom action to parse arguments as key=value pairs
+
+    Ensures that ``dest`` is a dict
     """
+
     def __call__(self, parser, namespace, values, option_string=None):
         # Make sure we have an empty dict rather than None
         if getattr(namespace, self.dest, None) is None:
@@ -35,7 +37,14 @@ class KeyValueAction(argparse.Action):
 
 
 class RangeAction(argparse.Action):
-    """A custom action to parse a single value or a range of values."""
+    """A custom action to parse a single value or a range of values
+
+    Parses single integer values or a range of integer values delimited
+    by a colon and returns a tuple of integers:
+    '4' sets ``dest`` to (4, 4)
+    '6:9' sets ``dest`` to (6, 9)
+    """
+
     def __call__(self, parser, namespace, values, option_string=None):
         range = values.split(':')
         if len(range) == 0: