Showing 1,558 of 1,565 total issues
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 ] )) ^
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.
Wrong continued indentation (add 4 spaces). Open
'(objectclass=GlueCE)',- Read upRead up
- Exclude checks
TODO '(objectclass=GlueCE)', ^ |
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...).
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...).
Variable name wh doesn't conform to snake_case naming style Open
wh += '"' + item + '",'- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Variable name wh doesn't conform to snake_case naming style Open
wh += '"' + item + '",'- 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
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 after 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 ] )) ^
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 os should be placed at the top of the module Open
import os- Read upRead up
- Exclude checks
Used when code and imports are mixed
Import from apel.db.records import JobRecord, SummaryRecord, NormalisedSummaryRecord, SyncRecord, CloudRecord, CloudSummaryRecord, StorageRecord should be placed at the top of the module Open
from apel.db.records import (JobRecord, SummaryRecord, NormalisedSummaryRecord,- Read upRead up
- Exclude checks
Used when code and imports are mixed
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)
Unnecessary pass statement Open
pass- Read upRead up
- Exclude checks
Used when a pass statement that can be avoided is encountered.
Consider using ternary (' AND '.join(parts) if len(parts) > 1 else parts[0]) Open
clauses = len(parts) > 1 and ' AND '.join(parts) or parts[0]- Read upRead up
- Exclude checks
Used when one of known pre-python 2.5 ternary syntax is used.
Wrong continued indentation (add 4 spaces). Open
Decimal(capability.split('=')[1])))- Read upRead up
- Exclude checks
TODO Decimal(capability.split('=')[1]))) ^ |
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...).
Variable name wh doesn't conform to snake_case naming style Open
wh = wh[:-1] + ')'- 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 __init__(self, db, qpath, inc_vos=None, exc_vos=None, local=False, withhold_dns=False):- Read upRead up
- Exclude checks
Used when a function or method takes too many arguments.
Too many local variables (22/15) Open
def fetch_specint(site, host='lcg-bdii.egi.eu', port=2170):- Read upRead up
- Exclude checks
Used when a function or method has too many local variables.