Showing 1,558 of 1,565 total issues
Variable name e
doesn't conform to snake_case naming style Open
except (KeyError, IndexError) as e:
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
standard import from decimal import Decimal, InvalidOperation
should be placed before import ldap
Open
from decimal import Decimal, InvalidOperation
- Read upRead up
- Exclude checks
Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)
No space allowed before bracket Open
buf.write('\n'.join( [ record.get_ur(self._withhold_dns) for record in records ] ))
- Read upRead up
- Exclude checks
Used when a wrong number of spaces is used around an operator, bracket or block opener. buf.write('n'.join( [ record.getur(self.withhold_dns) for record in records ] )) ^
Import import logging
should be placed at the top of the module Open
import logging
- Read upRead up
- Exclude checks
Used when code and imports are mixed
No space allowed before bracket Open
elems.append( column + RELATIONS[relation] + "'" + str(self.__dict__[elem]) + "'" )
- Read upRead up
- Exclude checks
Used when a wrong number of spaces is used around an operator, bracket or block opener.
elems.append( column + RELATIONS[relation] + '
+ str(self.dict[elem]) + '
)
^
Constant name logger
doesn't conform to UPPER_CASE naming style Open
logger = logging.getLogger(LOGGER_ID)
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Argument name db
doesn't conform to snake_case naming style Open
def __new__(cls, backend, host, port, username, pwd, db):
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Import outside toplevel (apel.db.backends.mysql) Open
from apel.db.backends.mysql import ApelMysqlDb
- Read upRead up
- Exclude checks
Used when an import statement is used anywhere other than the module toplevel. Move this import to the top of the file.
No space allowed before bracket Open
start_tuple = [ int(x) for x in start.split('-') ]
- Read upRead up
- Exclude checks
Used when a wrong number of spaces is used around an operator, bracket or block opener. start_tuple = [ int(x) for x in start.split('-') ] ^
No space allowed before bracket Open
buf.write('\n'.join( [ record.get_ur(self._withhold_dns) for record in records ] ))
- Read upRead up
- Exclude checks
Used when a wrong number of spaces is used around an operator, bracket or block opener. buf.write('n'.join( [ record.getur(self.withhold_dns) for record in records ] )) ^
Unable to import 'dirq.QueueSimple' Open
from dirq.QueueSimple import QueueSimple
- Read upRead up
- Exclude checks
Used when pylint has been unable to import a module.
Import from dirq.QueueSimple import QueueSimple
should be placed at the top of the module Open
from dirq.QueueSimple import QueueSimple
- Read upRead up
- Exclude checks
Used when code and imports are mixed
Wrong continued indentation (add 7 spaces). Open
ldap.SCOPE_SUBTREE,
- Read upRead up
- Exclude checks
TODO ldap.SCOPE_SUBTREE, ^ |
Missing class docstring Open
class ApelDbException(Exception):
- Read upRead up
- Exclude checks
Used when a class has no docstring.Even an empty class must have a docstring.
Unnecessary pass statement Open
pass
- Read upRead up
- Exclude checks
Used when a pass
statement that can be avoided is encountered.
No space allowed after bracket Open
buf.write('%%\n'.join( [ record.get_msg(self._withhold_dns) for record in records ] ))
- Read upRead up
- Exclude checks
Used when a wrong number of spaces is used around an operator, bracket or block opener. buf.write('%%n'.join( [ record.getmsg(self.withhold_dns) for record in records ] )) ^
Unable to import 'future.builtins' Open
from future.builtins import object, str
- Read upRead up
- Exclude checks
Used when pylint has been unable to import a module.
Import from apel.db import Query, ApelDbException, JOB_MSG_HEADER, SUMMARY_MSG_HEADER, NORMALISED_SUMMARY_MSG_HEADER, SYNC_MSG_HEADER, CLOUD_MSG_HEADER, CLOUD_SUMMARY_MSG_HEADER
should be placed at the top of the module Open
from apel.db import (Query, ApelDbException, JOB_MSG_HEADER, SUMMARY_MSG_HEADER,
- Read upRead up
- Exclude checks
Used when code and imports are mixed
Wrong continued indentation (add 4 spaces). Open
'(objectclass=GlueCE)',
- Read upRead up
- Exclude checks
TODO '(objectclass=GlueCE)', ^ |