Showing 1,558 of 1,565 total issues
Line too long (126/100) Open
for batch in self._db.get_records(record_type, table_name, 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.
Wrong continued indentation (add 4 spaces). Open
Decimal(capability.split('=')[1])))- Read upRead up
- Exclude checks
TODO Decimal(capability.split('=')[1]))) ^ |
Variable name fk doesn't conform to snake_case naming style Open
for fk in fks:- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
No space allowed after bracket Open
end_tuple = [ int(x) for x in end.split('-') ]- Read upRead up
- Exclude checks
Used when a wrong number of spaces is used around an operator, bracket or block opener. end_tuple = [ int(x) for x in end.split('-') ] ^
No space allowed before 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 ] )) ^
Unnecessary pass statement Open
pass- Read upRead up
- Exclude checks
Used when a pass statement that can be avoided is encountered.
Variable name wh doesn't conform to snake_case naming style Open
wh = '('- 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 (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.
standard import import os should be placed before from future import standard_library Open
import os- Read upRead up
- Exclude checks
Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)
Wrong continued indentation (add 4 spaces). Open
"NodeCount", "StartTime", "EndTime", "InfrastructureDescription", "InfrastructureType", "MemoryReal",- Read upRead up
- Exclude checks
TODO
NodeCount, StartTime, EndTime, InfrastructureDescription, InfrastructureType, MemoryReal,
^ |
Unnecessary parens after 'if' keyword Open
if (int(rc['Month']) == 12):- Read upRead up
- Exclude checks
Used when a single item in parentheses follows an if, for, or other keyword.
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.
Wrong continued indentation (remove 6 spaces). Open
"ResourceCapacityUsed", "LogicalCapacityUsed",- Read upRead up
- Exclude checks
TODO
ResourceCapacityUsed, LogicalCapacityUsed,
| ^
Too many statements (98/50) Open
def get_ur(self, withhold_dns=False):- Read upRead up
- Exclude checks
Used when a function or method has too many statements. You should then split it in smaller functions / methods.
Argument name ur doesn't conform to snake_case naming style Open
def _write_messages(self, record_type, table_name, query, ur):- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
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)
Wrong continued indentation (add 3 spaces). Open
"Site", "ValidFrom", "ValidUntil", "Processed"]- Read upRead up
- Exclude checks
TODO
Site, ValidFrom, ValidUntil, Processed]
^ |
Wrong continued indentation (add 4 spaces). Open
"VOGroup", "VORole", "WallDuration", "CpuDuration", "Processors",- Read upRead up
- Exclude checks
TODO
VOGroup, VORole, WallDuration, CpuDuration, Processors,
^ |
Line too long (125/100) Open
record_id = self.get_field('MachineName') + ' ' + self.get_field('LocalJobId') + ' ' + str(self.get_field('EndTime'))- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Variable name vo doesn't conform to snake_case naming style Open
role, group, vo = parse_fqan(self._record_content['FQAN'])- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).