Skip to content

Commit

Permalink
pylint: ignore possibly/used-before-assignment as it is prone to FP
Browse files Browse the repository at this point in the history
It does no control flow analysis, and upgrading to pylint 3.2.0 produced
many incorrect warnings.

Also ignore contextmanager-generator-missing-cleanup for now. It has FP
when there is no code after a yield (and thus no cleanup needs to be
handled), which is what we do. Currently under discussion upstream:
pylint-dev/pylint#9625

(cherry picked from commit e343590)
  • Loading branch information
eli-schwartz committed May 29, 2024
1 parent 2d807d5 commit c8be46e
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ disable=
cell-var-from-loop,
consider-using-f-string,
consider-using-with,
contextmanager-generator-missing-cleanup,
cyclic-import,
deprecated-decorator,
duplicate-code,
Expand Down Expand Up @@ -47,6 +48,7 @@ disable=
not-an-iterable,
not-callable,
pointless-string-statement,
possibly-used-before-assignment,
protected-access,
raise-missing-from,
redeclared-assigned-name,
Expand Down Expand Up @@ -75,6 +77,7 @@ disable=
unsubscriptable-object,
unused-argument,
unused-variable,
used-before-assignment,
useless-super-delegation,
wrong-import-order,
wrong-import-position,

0 comments on commit c8be46e

Please sign in to comment.