From a78d75f290278501b5d8130dc58420952a13e1a1 Mon Sep 17 00:00:00 2001 From: Dean Troyer Date: Thu, 26 Jun 2014 17:06:38 -0500 Subject: Fix PEP8 H405 errors Change-Id: Id9ea03e7d88148f84bffe1b18b5b4315e6123012 --- openstackclient/common/commandmanager.py | 9 ++++++--- openstackclient/common/parseractions.py | 15 ++++++++++++--- 2 files changed, 18 insertions(+), 6 deletions(-) (limited to 'openstackclient/common') 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: -- cgit v1.2.1