Showing 1,558 of 1,565 total issues
Variable name sl doesn't conform to snake_case naming style Open
(slt, sl) = self._check_factor(slt, sl)- 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 raise Open
if sfu == 'None':- Read upRead up
- Exclude checks
Used in order to highlight an unnecessary block of code following an if containing a raise statement. As such, it will warn when it encounters an else following a chain of ifs, all of them containing a raise statement.
standard import from xml.dom.minidom import Document should be placed before from future.builtins import str Open
from xml.dom.minidom import Document- 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 (101/100) Open
nwall.appendChild(doc.createTextNode('PT'+str(self.get_field('NormalisedWallDuration'))+'S'))- Read upRead up
- Exclude checks
Used when a line is longer than a given number of characters.
Too many local variables (23/15) Open
def get_ur(self):- Read upRead up
- Exclude checks
Used when a function or method has too many local variables.
standard import import logging should be placed before from future.builtins import object, str, zip Open
import logging- 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 time should be placed before from future.builtins import str 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)
standard import import logging should be placed before from future.builtins import str Open
import logging- 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 UR_OPEN doesn't conform to snake_case naming style Open
UR_OPEN = ('<sr:StorageUsageRecords xmlns:sr="http://eu-emi.eu/namespaces/2011/02/storagerecord">')- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Wrong continued indentation (add 14 spaces). Open
"CpuDuration", "NumberOfJobs"]- Read upRead up
- Exclude checks
TODO
CpuDuration, NumberOfJobs]
^ |
Argument name sf doesn't conform to snake_case naming style Open
def _check_factor(self, sfu, sf):- 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 (101/100) Open
# A trailing Z in the ISO format denotes UTC. We make this explicit for parsing.- 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(LOGGER_ID)- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Too many instance attributes (9/7) Open
class Record(object):- Read upRead up
- Exclude checks
Used when class has too many instance attributes, try to reduce this to get a simpler (and so easier to use) class.
Variable name dn doesn't conform to snake_case naming style Open
dn = self.get_field(Record.DN_FIELD)- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Wrong hanging indentation (remove 6 spaces). Open
EventRecord : 'CALL ReplaceEventRecord(%s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s)',- Read upRead up
- Exclude checks
TODO EventRecord : 'CALL ReplaceEventRecord(%s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s)', | ^
Variable name XML_HEADER doesn't conform to snake_case naming style Open
XML_HEADER = '<?xml version="1.0" ?>'- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Variable name XML_HEADER doesn't conform to snake_case naming style Open
XML_HEADER = '<?xml version="1.0" ?>'- Read upRead up
- Exclude checks
Used when the name doesn't conform to naming rules associated to its type (constant, variable, class...).
Import from apel.db.apeldb import ApelDb, Query, ApelDbException should be placed at the top of the module Open
from apel.db.apeldb import ApelDb, Query, ApelDbException- Read upRead up
- Exclude checks
Used when code and imports are mixed
standard import from datetime import datetime, timedelta should be placed before from future.builtins import str Open
from datetime import datetime, timedelta- Read upRead up
- Exclude checks
Used when PEP8 import order is not respected (standard imports first, then third-party libraries, then local imports)