Showing 1,562 of 1,569 total issues
Wrong continued indentation (add 1 space). Open
'CpuDuration' : lambda x: int(round(float(x[36]))*self._get_cpu_multiplier(x[1])),- Read upRead up
- Exclude checks
TODO 'CpuDuration' : lambda x: int(round(float(x[36]))*self.getcpu_multiplier(x[1])), ^|
Wrong continued indentation (add 1 space). Open
'MemoryVirtual' : lambda x: int(float(x[42])),- Read upRead up
- Exclude checks
TODO 'MemoryVirtual' : lambda x: int(float(x[42])), ^|
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...).
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...).
standard import import time should be placed before from apel.common import parse_time 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 cp doesn't conform to snake_case naming style Open
cp = re.compile(pattern)- 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 import iso8601 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)
standard import import re should be placed before import iso8601 Open
import re- Read upRead up
- Exclude checks
Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)
Cyclic import (apel.db -> apel.db.apeldb -> apel.db.backends.mysql -> apel.db.records -> apel.db.records.record) Open
"""A setup script for APEL.- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
Line too long (105/100) Open
'WallDuration' : lambda x: int(round(float(x[13]))*self._get_wall_multiplier(x[1])),- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Unnecessary pass statement Open
pass- Read upRead up
- Exclude checks
Used when a pass statement that can be avoided is encountered.
Wrong continued indentation (add 1 space). Open
'NodeCount' : lambda x: 0}- Read upRead up
- Exclude checks
TODO 'NodeCount' : lambda x: 0} ^|
Missing class docstring Open
class ParserException(Exception):- Read upRead up
- Exclude checks
Used when a class has no docstring.Even an empty class must have a docstring.
Line too long (102/100) Open
(\"([^"]|(""))*\") - we are looking for everything between " and " except for double quotation- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Exactly one space required after comma Open
parts = [x.split('=',1) for x in [y for y in self.LINE_EXPR.split(line) if len(y) > 1]]- Read upRead up
- Exclude checks
Used when a wrong number of spaces is used around an operator, bracket or block opener. parts = [x.split('=',1) for x in [y for y in self.LINE_EXPR.split(line) if len(y) > 1]] ^
standard import import re should be placed before from apel.db.records.blahd import BlahdRecord Open
import re- 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 (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.
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
Wrong continued indentation (add 1 space). Open
'Infrastructure' : lambda x: "APEL-CREAM-SGE",- Read upRead up
- Exclude checks
TODO
'Infrastructure' : lambda x: APEL-CREAM-SGE,
^|
Cyclic import (apel.db.records -> apel.db.records.sync) Open
"""A setup script for APEL.- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.