Showing 1,557 of 1,564 total issues
Import from .lsf import LSFParser
should be placed at the top of the module Open
from .lsf import LSFParser
- Read upRead up
- Exclude checks
Used when code and imports are mixed
Line too long (152/100) Open
end=1317534104 Exit_status=0 resources_used.cput=18:15:24 resources_used.mem=2031040kb resources_used.vmem=3335528kb resources_used.walltime=19:23:4
- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Cyclic import (apel.db.records -> apel.db.records.event) Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
Cyclic import (apel.db.records -> apel.db.records.processed) Open
"""A setup script for APEL.
- Read upRead up
- Exclude checks
Used when a cyclic import between two or more modules is detected.
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.
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.
Wrong continued indentation (add 1 space). Open
'Processors' : lambda x: procs,
- Read upRead up
- Exclude checks
TODO 'Processors' : lambda x: procs, ^|
Method name getTagByAttr
doesn't conform to snake_case naming style Open
def getTagByAttr(self, nodes, name, value, namespace=None):
- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Missing class docstring Open
class MultiplierError(Exception):
- Read upRead up
- Exclude checks
Used when a class has no docstring.Even an empty class must have a docstring.
Wrong continued indentation (add 1 space). Open
'LocalUserID' : lambda x: x[3],
- Read upRead up
- Exclude checks
TODO 'LocalUserID' : lambda x: x[3], ^|
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)
Wrong continued indentation (add 1 space). Open
'WallDuration' : lambda x: int(round(float(x[13]))*self._get_wall_multiplier(x[1])),
- Read upRead up
- Exclude checks
TODO 'WallDuration' : lambda x: int(round(float(x[13]))*self.getwall_multiplier(x[1])), ^|
Import from .blah import BlahParser
should be placed at the top of the module Open
from .blah import BlahParser
- Read upRead up
- Exclude checks
Used when code and imports are mixed
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])), ^|
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 fh
doesn't conform to snake_case naming style Open
fh = logging.FileHandler(logfile)
- 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 (125/100) Open
# 1007|cream_612883006|dteam005|dteam|2013-03-27T17:13:41|2013-03-27T17:13:44|00:00:03|3|prod|1|1|cert-40|||COMPLETED
- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Similar lines in 2 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.normalisedsummary:104
==apel.db.records.summary:103
try:
earliestend = rc['EarliestEndTime']
latestend = rc['LatestEndTime']
if not (monthstart <= earliestend <= monthend):
raise InvalidRecordException(EarliestEndTime is not within stated month.
)
if not (monthstart <= latestend <= month_end):
raise InvalidRecordException(LatestEndTime is not within stated month.
)
if earliestend > latestend:
raise InvalidRecordException(LatestEndTime is earlier than EarliestEndTime.
)
except TypeError:
# These two fields are not compulsory.
pass
# Check that the month isn't in the future
now = datetime.now()
if month_start > now:
raise InvalidRecordException(Month specified in record is in the future.
)
if not 1 <= int(self.recordcontent['Month']) <= 12:
raise InvalidRecordException(Month value is out of range
)
if int(self.recordcontent['WallDuration']) < 0:
raise InvalidRecordException(Negative WallDuration
)
if int(self.recordcontent['CpuDuration']) < 0:
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...).
Import from .htcondor import HTCondorParser
should be placed at the top of the module Open
from .htcondor import HTCondorParser
- Read upRead up
- Exclude checks
Used when code and imports are mixed