Showing 1,558 of 1,565 total issues
Constant name log
doesn't conform to UPPER_CASE naming style Open
log = logging.getLogger(__name__)
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Import from .sge import SGEParser
should be placed at the top of the module Open
from .sge import SGEParser
- Read upRead up
- Exclude checks
Used when code and imports are mixed
Unable to import 'future.builtins' Open
from future.builtins import zip
- Read upRead up
- Exclude checks
Used when pylint has been unable to import a module.
Cyclic import (apel.parsers -> apel.parsers.sge) Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
Method could be a function Open
def getText(self, nodes):
- Read upRead up
- Exclude checks
Used when a method doesn't use its bound instance, and so could be written as a function.
Line too long (111/100) Open
Parser is based on this specification: http://manpages.ubuntu.com/manpages/lucid/man5/sge_accounting.5.html
- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Line too long (109/100) Open
"/afs/cern.ch/user/r/raortega/prog/step3c/startEachset.pl 362 7 8" 277.210000 17.280000 0 0 -1 0 0 927804
- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Variable name rc
doesn't conform to snake_case naming style Open
rc = {}
- 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(__name__)
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Unnecessary else
after break
Open
if piece.lower().startswith('role='):
- Read upRead up
- Exclude checks
Used in order to highlight an unnecessary block of code following an if containing a break statement. As such, it will warn when it encounters an else following a chain of ifs, all of them containing a break statement.
Variable name md
doesn't conform to snake_case naming style Open
md = md5()
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Cyclic import (apel.db.records -> apel.db.records.storage) Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
Similar lines in 3 files Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Indicates that a set of similar lines has been detected among multiple file. This usually means that the code should be refactored to avoid this duplication. ==apel.db.records.job:220 ==apel.db.records.normalisedsummary:171 ==apel.db.records.summary:166 if self.getfield('VOGroup') is not None: vogroup = doc.createElement('urf:GroupAttribute') vogroup.setAttribute('urf:type', 'vo-group') vogroup.appendChild(doc.createTextNode(self.getfield('VOGroup'))) userid.appendChild(vogroup)
if self.getfield('VORole') is not None: vorole = doc.createElement('urf:GroupAttribute') vorole.setAttribute('urf:type', 'vo-role') vorole.appendChild(doc.createTextNode(self.getfield('VORole'))) user_id.appendChild(vorole)
Line too long (104/100) Open
'CpuDuration' : lambda x: int(round(float(x[36]))*self._get_cpu_multiplier(x[1])),
- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Constant name log
doesn't conform to UPPER_CASE naming style Open
log = logging.getLogger(__name__)
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Constant name log
doesn't conform to UPPER_CASE naming style Open
log = logging.getLogger(__name__)
- 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 apel.common import parse_time
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)
Constant name log
doesn't conform to UPPER_CASE naming style Open
log = logging.getLogger(__name__)
- 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 (115/100) Open
exec_host=lcg1277.gridpp.rl.ac.uk/5 Resource_List.cput=96:00:00 Resource_List.neednodes=lcg1277.gridpp.rl.ac.uk
- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Missing function or method docstring Open
def set_up_logging(logfile, level, console):
- Read upRead up
- Exclude checks
Used when a function or method has no docstring.Some special methods like init do not require a docstring.