diff options
| author | Simo Sorce <simo@redhat.com> | 2015-05-07 12:33:40 -0400 |
|---|---|---|
| committer | Rob Crittenden <rcritten@redhat.com> | 2015-05-07 14:44:20 -0400 |
| commit | 1bcc0d697dd37a9268641f0cbaa7e9e781552233 (patch) | |
| tree | f17e5fb0f9f0a1a922bcbd80ec798396c508f56d /ipsilon/util/errors.py | |
| parent | 93d4e52712767fe955f3a44a60a6c6f0f909423b (diff) | |
| download | ipsilon-1bcc0d697dd37a9268641f0cbaa7e9e781552233.tar.gz ipsilon-1bcc0d697dd37a9268641f0cbaa7e9e781552233.tar.xz ipsilon-1bcc0d697dd37a9268641f0cbaa7e9e781552233.zip | |
pylint 1.4.3 version fixes
Pylint 1.4.3 completely stopped recognizing the star-args condition.
In order to avoid pylint error with > 1.4.3 stop caring for star-args
and add cmdline option to ignore those errors completly so older pylint
versions are happy too.
Also fix type() vs isinstance() checks, isinstance is generally a more
correct approach to check for classes.
In some 'admin' files the type() -> isinstance() fix required to invert
the order in which ComplexList and MappingList are checked as the latter
is a subclass of ComplexList, so it needs to be checked first otherwise
the check for isinstance(option, ComplexList) matches for both and the
code stops functioning properly.
Signed-off-by: Simo Sorce <simo@redhat.com>
Reviewed-by: Rob Crittenden <rcritten@redhat.com>
Diffstat (limited to 'ipsilon/util/errors.py')
| -rw-r--r-- | ipsilon/util/errors.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/ipsilon/util/errors.py b/ipsilon/util/errors.py index 7017a1b..70d2da9 100644 --- a/ipsilon/util/errors.py +++ b/ipsilon/util/errors.py @@ -24,7 +24,6 @@ class Errors(Page): super(Errors, self).__init__(*args, **kwargs) def _error_template(self, *args, **kwargs): - # pylint: disable=star-args output_page = self._template(*args, **kwargs) # for some reason cherrypy will choke if the output # is a unicode object, so use str() here to please it |