Showing 1,557 of 1,564 total issues
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...).
Too many arguments (7/5) Open
def __new__(cls, backend, host, port, username, pwd, db):- Read upRead up
- Exclude checks
Used when a function or method takes too many arguments.
Line too long (106/100) Open
for batch in self._db.get_sync_records(query=query, records_per_message=self.records_per_message):- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
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 ] )) ^
Wrong continued indentation (add 7 spaces). Open
ldap.SCOPE_SUBTREE,- Read upRead up
- Exclude checks
TODO ldap.SCOPE_SUBTREE, ^ |
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.
Line too long (111/100) Open
UR_OPEN = ('<sr:StorageUsageRecords xmlns:sr="http://eu-emi.eu/namespaces/2011/02/storagerecord">')- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
standard import import datetime should be placed before from future import standard_library Open
import datetime- Read upRead up
- Exclude checks
Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)
Line too long (106/100) Open
earliest_text = time.strftime('%Y-%m-%dT%H:%M:%SZ', self.get_field('EarliestEndTime').timetuple())- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Too many statements (69/50) Open
def get_ur(self):- Read upRead up
- Exclude checks
Used when a function or method has too many statements. You should then split it in smaller functions / methods.
standard import import time should be placed before from future.builtins import str Open
import time- Read upRead up
- Exclude checks
Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)
Variable name ur doesn't conform to snake_case naming style Open
ur = doc.createElement('aur:SummaryRecord')- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Line too long (102/100) Open
raise InvalidRecordException('Invalid float value %s in field %s' % (value, name))- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Line too long (101/100) Open
nwall.appendChild(doc.createTextNode('PT'+str(self.get_field('NormalisedWallDuration'))+'S'))- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Wrong continued indentation (add 13 spaces). Open
" doesn't contain a float.")- Read upRead up
- Exclude checks
TODO
doesn't contain a float.)
^ |
Constant name log doesn't conform to UPPER_CASE naming style Open
log = 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...).
Wrong continued indentation (add 14 spaces). Open
"CpuDuration", "NumberOfJobs"]- Read upRead up
- Exclude checks
TODO
CpuDuration, NumberOfJobs]
^ |
Variable name dt doesn't conform to snake_case naming style Open
dt = datetime.utcfromtimestamp(time.mktime(time.strptime(dtval, isofmt)))- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Too many return statements (7/6) Open
def checked(self, name, value):- Read upRead up
- Exclude checks
Used when a function or method has too many return statement, making it hard to follow.