Custom changes

Custom checkers

EduLint provides custom checkers to report more problems frequent in novice code. It does so by extending Pylint with extra plugins. The checkers can therefore be invoked in the standard Pylint manner (e.g. using --enable or --disable) on the Edulint’s pylint option.

Improve For Loop

This section details messages emmited by the improve-for-loop checker.

Message name	Format	Description
use-foreach	Iterate directly: "for var in %s" (with appropriate name for "var")	Emitted when a for-range loop is used while a for-each loop would suffice.
use-enumerate	Iterate using enumerate: "for %s, var in enumerate(%s)" (with appropriate name for "var")	Emitted when a for-range loop is used with the element at each index is accessed as well.

Long Code

This section details messages emmited by the long-code checker.

Message name	Format	Description
long-script	Too much code outside of functions or classes (%d which is over %d statements).	Emitted when there are too many lines of code on the top level that are not import or function or class definition.
long-function	Function '%s' is too long (%d which is over %d statements).	Emitted when there are too many statements inside a function definition.

No Global Variables

This section details messages emmited by the no-global-variables checker.

Message name	Format	Description
no-global-vars	Do not use global variables; you use %s, modifying it for example at line %i.	Emitted when the code uses global variables.

No Duplicate Code

This section details messages emmited by the no-duplicate-code checker.

Message name	Format	Description
duplicate-if-branches	Identical code inside all if's branches, move %d lines %s the if.	Emitted when identical code starts or ends all branches of an if statement.
duplicate-seq-ifs	Identical code inside %d consecutive ifs, join their conditions using 'or'.	Emitted when several consecutive if statements have identical bodies and thus can be joined by or in their conditions.
duplicate-exprs	A complex expression '%s' used repeatedly (on lines %s). Extract it to a local variable or create a helper function.	Emitted when an overly complex expression is used multiple times.
duplicate-blocks	Duplicate blocks starting on lines %s. Extract the code to a helper function.	Emitted when there are duplicate blocks of code as a body of an if/elif/else/for/while/with/try-except block.
duplicate-sequence	Duplicate sequence of %d repetitions of %d lines of code. Use a loop to avoid this.	Emitted when there is a sequence of similar sub-blocks inside a block that can be replaced by a loop.

Improper Loop

This section details messages emmited by the improper-loop checker.

Message name	Format	Description
no-while-true	The while condition can be replaced with '<negated %s>'	Emitted when the condition of a while loop is ‘True’ unnecessarily.
use-tighter-boundaries	Use tighter range boundaries, the %s iteration never happens.	Emitted when the boundaries of a range can be made tighter.
modifying-iterated-structure	Iterated structure %s is being modified inside the for loop body. Use while loop or iterate over a copy.	Emitted when the structure that is being iterated over is being modified in the for loops body.
changing-control-variable	Changing the control variable %s of a for loop has no effect.	Emitted when the control variable of a for loop is being changed.
use-for-loop	Use for loop.	Emitted when a while loop can be transformed into a for loop.
loop-shadows-control-variable	Inner for loop shadows outer for loop's control variable %s.	Emitted when a for loop shadows control variable of an outer for loop.

Invalid_For_Target

This section details messages emmited by the invalid_for_target checker.

Message name	Format	Description
invalid-for-target	For loop or comprehension variable "%s" should not be a part of a larger object.	Used when you have an index variable in a for loop or comprehensionthat is in subscript or object attribute form

One_Iteration

This section details messages emmited by the one_iteration checker.

Message name	Format	Description
one-iteration	This loop will only ever run for one iteration	Reported when the loop body always ends the loop in its first iteration (e.g., by returning or using the “break” keyword).

Shadowing_In_Comprehension

This section details messages emmited by the shadowing_in_comprehension checker.

Message name	Format	Description
shadowing-in-comprehension	Comprehension variable '%s' shadows a variable in an outer scope	Used when there is shadowing inside a comprehension

Top_Level_Code

This section details messages emmited by the top_level_code checker.

Message name	Format	Description
forbidden-top-level-code	Forbidden top-level code found on line %s	Used when you write top-level code that is not allowed. The allowed top-level code includes imports, definitions, and assignments.

Short Problems

This section details messages emmited by the short-problems checker.

Message name	Format	Description
use-append	Use %s.append(%s) instead of %s.	Emitted when code extends list by a single argument instead of appending it.
use-integral-division	Use integral division //.	Emitted when the code uses float division and converts the result to int.
use-isdecimal	Use isdecimal to test if string contains a number.	Emitted when the code uses isdigit or isnumeric.
no-loop-else	Do not use %s loop with else.	Emitted when the code contains loop with else block.
use-elif	Use elif.	Emitted when the code contains else: if construction instead of elif. Might return a false positive if the code mixes tabs and spaces.
at-most-one-iteration-for-loop	The for loop makes %s.	Emitted when a for loop would always perform at most one iteration.
no-repeated-op	Use %s instead of repeated %s in %s.	Emitted when the code contains repeated adition/multiplication instead of multiplication/exponentiation.
redundant-arithmetic	Redundant arithmetic: %s	Emitted when there is redundant arithmetic (e.g. +0, *1) in an expression.
use-augmented-assign	Use augmented assignment: '%s %s= %s'	Emitted when an assignment can be simplified by using its augmented version.
do-not-multiply-mutable	Do not multiply list with mutable content.	Emitted when a list with mutable contents is being multiplied.
redundant-elif	Use else instead of elif.	Emitted when the condition in elif is negation of the condition in the if.
unreachable-else	Unreachable else.	Emitted when the else branch is unreachable due to totally exhaustive conditions before.
no-is-bool	Use '%s' directly rather than as '%s'.	Emitted when the is operator is used with a bool value.
use-ord-letter	Use "%s" instead of using the magical constant %i.	Emitted when the code uses a magical constant instead of a string literal.
use-literal-letter	Remove the call to 'ord' and compare to the string directly "%s" instead of using the magical constant %i. Careful, this may require changing the comparison operator.	Emitted when the code uses a magical constant instead of a string literal in a comparison.
use-early-return	Use early return.	Emitted when a long block of code is followed by an else that just returns, breaks or continues.

Simplifiable If

This section details messages emmited by the simplifiable-if checker.

Message name	Format	Description
simplifiable-if-return	The if statement can be replaced with 'return %s'	Emitted when the condition of an if statement can be returned directly (possibly negated).
simplifiable-if-return-conj	The if statement can be replaced with 'return %s'	Emitted when the condition of an if statement and the returned values can be combined using logical operators.
simplifiable-if-assignment	The conditional assignment can be replace with '%s = %s'	Emitted when the condition of an if statement can be assigned directly (possibly negated).
simplifiable-if-assignment-conj	The conditional assignment can be replace with '%s = %s'	Emitted when the condition of an if statement and the assigned values can be combined using logical operators.
simplifiable-if-expr	The if expression can be replaced with '%s'	Emitted when the condition of an if expression can be returned directly (possibly negated).
simplifiable-if-expr-conj	The if expression can be replaced with '%s'	Emitted when the condition of an if expression and the returned values can be combined using logical operators.
simplifiable-if-pass	Use 'if %s: <else body>' instead of 'pass'	Emitted when there is an if condition with a pass in the positive branch.
no-value-in-one-branch-return	Both branches should return a value explicitly (one returns implicit None)	Emitted when one branch returns a value and the other just returns.
simplifiable-if-nested	The if statement can be merged with the nested one to 'if %s:'	Emitted when the condition of an if statement can be merged with its nested if’s condition using logical operators.
simplifiable-if-seq	The if statement can be merged with the following one to 'if %s:'	Emitted when the condition of an if statement can be merged with the next if’s condition using logical operators.

Tweakers