summaryrefslogtreecommitdiffstats
path: root/make-lint
diff options
context:
space:
mode:
authorJan Cholasta <jcholast@redhat.com>2011-11-21 10:50:27 -0500
committerMartin Kosek <mkosek@redhat.com>2011-11-30 17:08:35 +0100
commit135ccf89de866ea2cdda96993ed2743394e1e716 (patch)
tree1b3c7bef4d5653255b75014218d1d0be9b10d5bb /make-lint
parent2ac9d4816a85822825257e16f4fcf74e15a8ea02 (diff)
downloadfreeipa-135ccf89de866ea2cdda96993ed2743394e1e716.tar.gz
freeipa-135ccf89de866ea2cdda96993ed2743394e1e716.tar.xz
freeipa-135ccf89de866ea2cdda96993ed2743394e1e716.zip
Parse comma-separated lists of values in all parameter types. This can be enabled for a specific parameter by setting the "csv" option to True.
Remove "List" parameter type and replace all occurences of it with appropriate multi-valued parameter ("Str" in most cases) with csv enabled. Add new parameter type "Any", capable of holding values of any type. This is needed by the "batch" command, as "Str" is not suitable type for the "methods" parameter. ticket 2007
Diffstat (limited to 'make-lint')
-rwxr-xr-xmake-lint42
1 files changed, 21 insertions, 21 deletions
diff --git a/make-lint b/make-lint
index 59c472221..b5df9e4a6 100755
--- a/make-lint
+++ b/make-lint
@@ -43,30 +43,30 @@ IGNORE_PATHS = ('build', 'dist', 'install/po/test_i18n.py', 'lite-server.py',
class IPATypeChecker(TypeChecker):
# 'class': ('generated', 'properties',)
ignore = {
- 'ipalib.base.NameSpace': ('find',),
- 'ipalib.cli.Collector': ('__options',),
- 'ipalib.config.Env': ('*'),
- 'ipalib.plugable.API': ('Command', 'Object', 'Method', 'Property',
- 'Backend', 'log', 'plugins'),
- 'ipalib.plugable.Plugin': ('Command', 'Object', 'Method', 'Property',
+ 'ipalib.base.NameSpace': ['find'],
+ 'ipalib.cli.Collector': ['__options'],
+ 'ipalib.config.Env': ['*'],
+ 'ipalib.plugable.API': ['Command', 'Object', 'Method', 'Property',
+ 'Backend', 'log', 'plugins'],
+ 'ipalib.plugable.Plugin': ['Command', 'Object', 'Method', 'Property',
'Backend', 'env', 'debug', 'info', 'warning', 'error', 'critical',
- 'exception', 'context', 'log'),
- 'ipalib.plugins.baseldap.CallbackInterface': ('pre_callback',
- 'post_callback', 'exc_callback'),
- 'ipalib.plugins.misc.env': ('env',),
- 'ipalib.parameters.Param': ('cli_name', 'cli_short_name', 'label',
+ 'exception', 'context', 'log'],
+ 'ipalib.plugins.baseldap.CallbackInterface': ['pre_callback',
+ 'post_callback', 'exc_callback'],
+ 'ipalib.plugins.misc.env': ['env'],
+ 'ipalib.parameters.Param': ['cli_name', 'cli_short_name', 'label',
'doc', 'required', 'multivalue', 'primary_key', 'normalizer',
'default', 'default_from', 'create_default', 'autofill', 'query',
- 'attribute', 'include', 'exclude', 'flags', 'hint', 'alwaysask'),
- 'ipalib.parameters.Bool': ('truths', 'falsehoods'),
- 'ipalib.parameters.Int': ('minvalue', 'maxvalue'),
- 'ipalib.parameters.Float': ('minvalue', 'maxvalue'),
- 'ipalib.parameters.Data': ('minlength', 'maxlength', 'length',
- 'pattern', 'pattern_errmsg'),
- 'ipalib.parameters.Enum': ('values',),
- 'ipalib.parameters.List': ('separator', 'skipspace'),
- 'ipalib.parameters.File': ('stdin_if_missing'),
- 'urlparse.SplitResult': ('netloc',),
+ 'attribute', 'include', 'exclude', 'flags', 'hint', 'alwaysask',
+ 'sortorder', 'csv', 'csv_separator', 'csv_skipspace'],
+ 'ipalib.parameters.Bool': ['truths', 'falsehoods'],
+ 'ipalib.parameters.Int': ['minvalue', 'maxvalue'],
+ 'ipalib.parameters.Float': ['minvalue', 'maxvalue'],
+ 'ipalib.parameters.Data': ['minlength', 'maxlength', 'length',
+ 'pattern', 'pattern_errmsg'],
+ 'ipalib.parameters.Enum': ['values'],
+ 'ipalib.parameters.File': ['stdin_if_missing'],
+ 'urlparse.SplitResult': ['netloc'],
}
def _related_classes(self, klass):